-
Notifications
You must be signed in to change notification settings - Fork 2.3k
misc/gamtor.cpp: Add some system base ROMs [ClawGrip] #14732
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
base: master
Are you sure you want to change the base?
Conversation
src/mame/misc/gamtor.cpp
Outdated
|
|
||
| // Money Controls Universal Hopper model MK4, standard on the Gaminator cabinet. | ||
| // TODO: Needs to be moved to own file | ||
| class mc_mk4_hopper_device : public ticket_dispenser_device |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don’t know what you’re trying to do, but making this derive from the ticket dispenser device isn’t the way to do it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've derived from hopper_device instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What I'm trying to do is create a Hopper class for a specific hopper model, so it can serve as a placeholder for the dumped ROM and it can be preserved.
Thx!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What he means is, don't derive it from that class at all. The generic hopper/ticket device is meant to be the mechanical part and sensor. If there's hopper hardware with an MCU controlling it, that can be its own device, with a machine_config that adds HOPPER.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahhhh... Ok, done!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it OK now? Thanks!
No description provided.