heather162/scummvm — comment on commit 1cded6587635a791d56d08c54abc116ab3bdba33
File:  line 0
By heather162:

Oops, yes, the check is wrong, that's why it's nice to have somebody else look at it. :)

The behavior I am seeing is that without this change none of the sounds are playing (except for video and 1 sound on the restore screen). If I use the in game debugger to view songlib, sound1-sound6 are listed but they all have a resource id of 0. Tracing through the code I found that when the SoundManager tries to find an open sound slot they are all reported as full because their handle is non-zero. The 1 sound that does play on the restore screen uses an additional entry called oldSound. What other information would be helpful?