Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix typo 8xyE to 8xy6 #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JanPoonthong
Copy link

@JanPoonthong JanPoonthong commented Apr 22, 2021

You got the test case opposite.
8xy6: http://devernay.free.fr/hacks/chip8/C8TECH10.HTM#8xy6
8xyE http://devernay.free.fr/hacks/chip8/C8TECH10.HTM#8xyE\

I didn't know how to generate the .ch8 file, please generate new file for .ch8 too.

Love your test cases.

The test case was via
@JanPoonthong JanPoonthong mentioned this pull request Apr 22, 2021
Copy link

@Timendus Timendus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that you are right, and these instructions are the wrong way around. I've suggested a code change that will also swap the 86 and 8E on screen.

@@ -156,15 +156,15 @@ v7 := 120
v7 -= v6
if v7 == 236 then i := imageok
sprite x2 y 4
#test 8xy6
#test 8xyE
y := 6
drawop im8 im6

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
drawop im8 im6
drawop im8 imE

y := 6
drawop im8 im6
i := imagefalse
v6 := 224
v6 <<= v6
if v6 == 192 then i := imageok
sprite x2 y 4
#test 8xyE
#test 8xy6
y := 11
drawop im8 imE

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
drawop im8 imE
drawop im8 im6

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants