Viewing Issue Advanced Details
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
08668 Misc. Minor Always Jun 17, 2023, 17:21 Jun 25, 2023, 05:12
Tester Balthier View Status Public Platform MAME (Official Binary)
Assigned To hap Resolution Fixed OS Windows 10/11 (64-bit)
Status [?] Resolved Driver
Version 0.255 Fixed in Version 0.256 Build 64-bit
Fixed in Git Commit efa7cb3 Github Pull Request #
Summary 08668: All sets in segas16a.cpp: Coin counters increase at each boot up even though credits are not being added
Description When running a game, the coin counters numbers in the machine's cfg file will increase by 1 on each start up even though credits are not being added by the user.
Steps To Reproduce 1. run any of the affected games ( http://adb.arcadeitalia.net/lista_mame.php?game_sourcefile=segas16a.cpp ).
2. let the game run for a little bit, just don't add any credits.
3. exit the game, go to your cfg directory and examine the game's cfg file. The coin counter numbers should be at 1 now.
4. run the same game and repeat step 2.
5. repeat step 3, and the counters will now be at 2.
Additional Information Example sjryuko1.cfg after one run, no credits added:
<?xml version="1.0"?>
<!-- This file is autogenerated; comments and unknown tags will be stripped -->
<mameconfig version="10">
    <system name="sjryuko1">
        <counters>
            <coins index="0" number="1" />
            <coins index="1" number="1" />
        </counters>
    </system>
</mameconfig>

This one started with Mame version 0.147, prior versions don't autoincrement.

A side note, I tested the 6 key and it adds coins to the second counter if the coin dip switches are changed a bit, so at least that works as hap explained on another report.
Github Commit
Flags
Regression Version
Affected Sets / Systems All sets in segas16a.cpp
Attached Files
 
Relationships
There are no relationship linked to this issue.
Notes
1
User avatar
No.21565
hap
Developer
Jun 17, 2023, 17:45
It's an 8255 tristate issue again.
I think it makes more sense if it's default 0, though it'll cause a few regressions (that can be fixed).

I already tried it once: https://github.com/mamedev/mame/commit/716d06341dba3ab9ee883ce82abe1d4fe8202b8b
got reverted here: https://github.com/mamedev/mame/commit/e53b995a475a995420d19fe08aa3f70d74b404f6