search

MiSTer-devel / NeoGeo_MiSTer

NeoGeo for MiSTer
GNU General Public License v2.0
146 stars 78 forks source link

Extended sustain (Sound) on Money Idol Exchanger samples. #67

Closed pepinw closed 3 years ago

pepinw commented 4 years ago

I'll admit there have been many improvements in the NeoGeo cores, but this one seems more like a regression.

Core version: 19.12.05

Play any mode of the game, or watch the attract loop.

You'll here when the coins connect in the field, there's a ... WAY longer sustain on the coin connect sound, which takes it from a short "clink" to a long "beeeeeeeeeeeeeeeeeeeeeeeep" --

In the previous version of the core (or jt's sound core) this did not happen.

pepinw commented 4 years ago

I decided to try older cores to figure out when the error happed in the build tree.

19.09.11 had the bug, the previous revision 19.09.07 does not have the bug.. so, it was introduced after that checkpoint.

raygan commented 4 years ago

This is probably the same as the issue I submitted: https://github.com/MiSTer-devel/NeoGeo_MiSTer/issues/56 I first tried the core around November, so I didn't realize it was a regression.

KKnot007 commented 4 years ago

Same here...coin sounds are creepy in the last core version

sorgelig commented 4 years ago

added the commit fixing this issue

M-Walrus commented 3 years ago

This issue was fixed and should be closed.

As noted above, this is a duplicate of issue #56 , which was already closed out.