Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes issue reported in #3700 (comment).
86box seems to have a more correct ESS mixer implementation, although it's unclear whether that's accurate to the real hardware or they just guessed the values.
https://github.com/86Box/86Box/blob/e61c6205584e4414fc4a64494be5c6f1aab3e6ff/src/sound/snd_sb.c#L93
In any case, reconciling the ESS and SB-specific mixer volume handling would take some non-trivial effort, and I'm not going to complicate our Sound Blaster code with that for the sake of a single 2024 demo. All in all, it's a rather insignificant detail to get the volume curve exactly right.
Manual testing
Now you can set the audio volume with the
+
and-
keys in koolnESS: https://www.pouet.net/prod.php?which=96920Checklist
Please tick the items as you have addressed them. Don't remove items; leave the ones that are not applicable unchecked.
I have: