Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Tetris ROM does not run #1

Closed
ghost opened this issue Sep 15, 2017 · 3 comments
Closed

Tetris ROM does not run #1

ghost opened this issue Sep 15, 2017 · 3 comments
Assignees

Comments

@ghost
Copy link

ghost commented Sep 15, 2017

[Cartridge.cpp:50] Reading ROM from path: Tetris (USA).nes [Cartridge.cpp:68] Reading header, it dictates: [Cartridge.cpp:71] 16KB PRG-ROM Banks: 2 [Cartridge.cpp:79] 8KB CHR-ROM Banks: 2 [Cartridge.cpp:82] Name Table Mirroring: 0 [Cartridge.cpp:85] Mapper #: 1 [Cartridge.cpp:88] Extended (CPU) RAM: false [Cartridge.cpp:102] ROM is NTSC compatible. [MapperSxROM.cpp:29] Using CHR-ROM [PictureBus.cpp:94] Unsupported Name Table mirroring : 1162765676 [CPU.cpp:138] Unrecognized opcode: 2 [CPU.cpp:138] Unrecognized opcode: 2 [CPU.cpp:138] Unrecognized opcode: 42 [CPU.cpp:138] Unrecognized opcode: b2 [CPU.cpp:138] Unrecognized opcode: 5a [CPU.cpp:138] Unrecognized opcode: 42
There is a gray screen in the main window.

amhndu added a commit that referenced this issue Sep 15, 2017
Erraneous logic where the bit was being set instead of unset
@amhndu
Copy link
Owner

amhndu commented Sep 15, 2017

Thanks for reporting the bug!
It has been fixed

@amhndu amhndu closed this as completed Sep 15, 2017
@amhndu amhndu self-assigned this Sep 15, 2017
@amhndu
Copy link
Owner

amhndu commented Sep 15, 2017

Can you confirm if this works for you ?

@amhndu amhndu reopened this Sep 15, 2017
@ghost
Copy link
Author

ghost commented Sep 15, 2017

Yes, it works excellently. It's a very good emulator. I think if you added a key config menu into the window (instead of a configuration menu) that would be great, but I know that it's probably really hard. (I can't understand the source code, but that's because I only know basic C++)

@ghost ghost closed this as completed Sep 15, 2017
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant