Bug in windows opcoder
CodeMaxx opened this issue · 5 comments
CodeMaxx commented
Only one of line 15 and line 63 can be correct.
@Pratik151 @Ali-Razmjoo
Pratik151 commented
@CodeMaxx line 63 is correct. you can remove line 15 one. Make PR if you want as you reported :) or I will make PR if you can't. Remove it for linux opcoder also.
Thanks for reporting
Ali-Razmjoo commented
@CodeMaxx Thanks for your report, @Pratik151 yes that's correct, opcode of pop %eax
must be 58
.
Ali-Razmjoo commented
@Pratik151 Please send the fixation. thanks.
Pratik151 commented
@Ali-Razmjoo I asked Akash if he wants to do it as he reported it :P and he said that he will do it yesterday. I will ask him if he has done or I will make PR
Ali-Razmjoo commented
Thanks, merged.