05083: mjflove, hginga: Sound is not restored

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
05083	Save/Restore	Minor	Always	Dec 15, 2012, 11:17	15 days ago

Tester	psx_	View Status	Public	Platform	MAME (Official Binary)
Assigned To		Resolution	Invalid report	OS	Windows Vista/7 (64-bit)
Status [?]	Closed			Driver	dynax/ddenlovr.cpp
Version	0.139	Fixed in Version		Build	Normal
		Fixed in Git Commit		Github Pull Request #

Summary	05083: mjflove, hginga: Sound is not restored
Description	I'm sorry in poor English. From the time that corresponds to the state saving perhaps, mjflove, hginga sound is not restored when you load a save file. These common point is that you are using a macro to load the sound ROM_RELOAD rom.
Steps To Reproduce
Additional Information	If you attempt to load the save state immediately after starting the game, the sound will not come back correctly and just play a stuck note or decay to silence. It will work again as soon as there is a change in the music. (I saved state in the title sequence, then restored it immediately. It plays a stuck note until after the title screen or when a coin is inserted.) If you wait until the game starts playing sound again, then load the save state, it will continue playing whatever it was playing when you loaded the savestate until there is a change in the music. (I saved the state in a game, then loaded the save state on the title sequence. The title sequence music continued to play until I lost, when the "You Lose" jingle played.)
Github Commit

Flags
Regression Version
Affected Sets / Systems	mjflove, hginga

Attached Files

No.09135 NekoEd Senior Tester Dec 15, 2012, 14:34 edited on: Dec 15, 2012, 14:49	Confirmed, added additional information.
No.09137 psx_ Tester Dec 15, 2012, 17:23	Thank you for your follow-up!
No.22537 hap Developer 15 days ago	ddenlovr.cpp doesn't officially support savestates. Expect bugs to happen if you load/save state anyway, that goes for any driver that hasn't added support for savestates.
No.22541 Osso Moderator 15 days ago	tbf to the original poster, it did when they created the report. The flag was removed in 2015. https://git.redump.net/mame/commit/?id=a3e1a6ccc8b48f0a62ffa7ae08c62e311399e00c