From 9493b9add55e1254ad56b4ffdf8b076b22c08546 Mon Sep 17 00:00:00 2001 From: Filippos Karapetis Date: Wed, 24 Nov 2010 14:51:29 +0000 Subject: [PATCH] SCI: Moved handling of special reverb value 127 outside of the MIDI driver svn-id: r54459 --- engines/sci/sound/drivers/midi.cpp | 4 ++-- engines/sci/sound/midiparser_sci.cpp | 3 ++- engines/sci/sound/music.cpp | 7 ++++++- 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/engines/sci/sound/drivers/midi.cpp b/engines/sci/sound/drivers/midi.cpp index 70e1cf21330..ccb1c87d027 100644 --- a/engines/sci/sound/drivers/midi.cpp +++ b/engines/sci/sound/drivers/midi.cpp @@ -378,10 +378,10 @@ int MidiPlayer_Midi::getVolume() { } void MidiPlayer_Midi::setReverb(byte reverb) { - assert(reverb < kReverbConfigNr || reverb == 127); + assert(reverb < kReverbConfigNr); _reverb = reverb; - if (_hasReverb && _reverb != 127) // 127: SCI invalid, don't send to sound card + if (_hasReverb) sendMt32SysEx(0x100001, _reverbConfig[_reverb], 3, true); } diff --git a/engines/sci/sound/midiparser_sci.cpp b/engines/sci/sound/midiparser_sci.cpp index abd3894d9a1..398917764ca 100644 --- a/engines/sci/sound/midiparser_sci.cpp +++ b/engines/sci/sound/midiparser_sci.cpp @@ -501,7 +501,8 @@ void MidiParser_SCI::parseNextEvent(EventInfo &info) { // Also, sci/sound/iterator/iterator.cpp, function BaseSongIterator::parseMidiCommand() switch (info.basic.param1) { case kSetReverb: - ((MidiPlayer *)_driver)->setReverb(info.basic.param2); + if (info.basic.param2 != 127) // 127: SCI invalid, ignore + ((MidiPlayer *)_driver)->setReverb(info.basic.param2); break; case kMidiHold: // Check if the hold ID marker is the same as the hold ID diff --git a/engines/sci/sound/music.cpp b/engines/sci/sound/music.cpp index 010f161a7a0..6628c175c63 100644 --- a/engines/sci/sound/music.cpp +++ b/engines/sci/sound/music.cpp @@ -227,7 +227,12 @@ MusicEntry *SciMusic::getActiveSci0MusicSlot() { void SciMusic::setReverb(byte reverb) { Common::StackLock lock(_mutex); - _pMidiDrv->setReverb(reverb); + if (reverb != 127) // 127: SCI invalid, ignore + _pMidiDrv->setReverb(reverb); + + // SSCI stored a separate reverb value per song + // We don't, currently, as the current functionality + // works without an additional variable } byte SciMusic::getReverb() {