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

galaxian/galaxian.cpp: Add another Video Dens Pac-Man bootleg #12301

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

clawgrip
Copy link
Contributor

New working clone

Pac-Man (Video Dens, Spanish bootleg on Galaxian hardware, set 2) [Arcadedealer aka Edgar Solé aka Dios, Recreativas.org]

New working clone
---------------------
Pac-Man (Video Dens, Spanish bootleg on Galaxian hardware, set 2) [Arcadedealer aka Edgar Solé aka Dios, Recreativas.org]
@clawgrip
Copy link
Contributor Author

clawgrip commented Apr 23, 2024

ROM 1 only differs on just one byte from the existing set (pacmanblv). Can anyone please check if this seems like a legit change and is not a bit rot?

@Osso13
Copy link
Member

Osso13 commented Apr 24, 2024

The only difference is byte 0x14d which is 0x40 in the old dump and 0x00 here. Looking at the disassembly, it seems to be doing a series of accesses to RAM which starts at 0x4000.
immagine
It would be strange that the bootleggers intentionally changed only one of those accesses to point to ROM area, so I'd say the old dump is correct.

And reverted byte 0x14d to 0x40, like "pacmanblv"
@clawgrip
Copy link
Contributor Author

Thanks a lot!

I've modified it to use ROM 1 from "pacmanblv" and marked it as BAD_DUMP.

@cuavas cuavas merged commit 2778f9e into mamedev:master Apr 24, 2024
0 of 5 checks passed
@clawgrip clawgrip deleted the patch-3 branch April 24, 2024 18:49
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

Successfully merging this pull request may close these issues.

None yet

3 participants