search

libretro / gpsp

gpSP for libretro.
GNU General Public License v2.0
52 stars 52 forks source link

Fix Noise Channel (Add Noise handling code from TempGBA) #203

Closed andymcca closed 1 year ago

andymcca commented 1 year ago

A user on RGH Discord group pointed out that certain sound effects were missing. (Sonic Advance was used as an example, when jumping on a box or enemy). After investigating with the user, we found that it was specifically noise sound effects on GBC Sound Channel 4 that were missing.

I tested the ROM on TempGBA and found the sound was correctly emulated. I have therefore implemented the differences for noise sample handling from TempGBA to this fork of gpsp in this PR. I have tested Sonic Advance and it fixes the issue (i.e. a proper 'crunch' sound is now played when jumping on a box, which is correct behaviour).

andymcca commented 1 year ago

Closing and will open a new PR with the same effect and less code changes.