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

Ms. Pacman Twin #126

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Ms. Pacman Twin #126

merged 1 commit into from
Apr 10, 2024

Conversation

arcadez2003
Copy link
Contributor

  • Added support for rare, Ms. Pacman Twin (mspactwin) simultaneous 2 player play [arcadez, grant2258, mahoneyt944]

* Added support for rare, Ms. Pacman Twin (mspactwin) simultaneous 2 player play [arcadez, grant2258, mahoneyt944]
@arcadez2003
Copy link
Contributor Author

arcadez2003 commented Mar 28, 2024

@amadvance i was clearing out some dev code i had kicking around and stumbled upon a part converted driver i
must have been working on some time back which adds support for Ms. Pacman Twin a recently dumped hack
which turns the game into a simultaneous two player version.

So i finished off the driver and here it is updated for this core im not 100% sure this will work i have some concerns
around the memory mapping but rather than rabbiting on about it now try the game and lemmie know if it works
and if not we can cross that bridge when it comes.

This is a good wee version of a classic game as two players can be used at the same time it was high on the
most wanted list for years before a dump finally turned up so it's well worth adding here.

Regards.

EDIT dont worry about the build failure it's related as usual to me pushing each file one at a time
any errors around mspactwin_videoram_w will be sorted when all files are merged.

@amadvance amadvance merged commit 9214026 into amadvance:master Apr 10, 2024
1 check failed
@amadvance
Copy link
Owner

Hi @arcadez2003
It looks like that we miss the definition of memory_set_opcode_base() in MACHINE_RESET( mspactwin ). At present, it doesn't build, even adding all the following pull requests.
Can you please check?

@arcadez2003
Copy link
Contributor Author

@amadvance damn i forgot that MAME totally changed the CPU decryption handling in the 101.u3 release, so long
story short the decryption code will need to be updated to work within your MAME106 codebase and it's not my
area of expertise unfortunately.

But i've asked @grant2258 who originally reworked the Ms Pacman Twin decryption for use in MAME78/84 if he
could help us with this so hang tight for now.

@grant2258
Copy link
Contributor

Been away for a bit will have a look when I get the chance sorry about the delay just got home.

@amadvance
Copy link
Owner

Thanks @grant2258, and don't worry. Take your time.

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.

3 participants