id summary reporter owner description type status priority component version resolution keywords cc game 9261 MT-32 reverb config parsed incorrectly SF/sven3000 bluegr "{{{#!Markdown ScummVM doesn't correctly parse the reverb data in the MT-32 patch resource. It also doesn't send the default reverb settings to the MT-32. This problem seems to apply to every SCI0 and SCI1 game. In the MidiPlayer\_Midi::readMt32Patch \(engines/sci/sound/driver/midi.cpp\) function, the \_reverbConfig array is initialized in the wrong order. It is read row-wise while it is stored column-wise. The attached patch includes a fix for this function. In addition, the reverb settings stored in \_reverbConfig\[0\] need to be sent to the MT-32 somewhere. The real SCI engine \(observed by running the games in DOSBox\) sends this command before it first plays music. I think the best place to put it would probably be MidiParser\_SCI::sendInitCommands \(engines/sci/sound/midiparser\_sci.cpp\), but that seems to send it more often than the real SCI does. If you know a better place to put it, be my guest. Because I wasn't sure where to put it, the attached patch does NOT include this\! It only fixes the reverb config parsing\! This behaviour was observed with ScummVM 1.2.0 and svn revision 54451, on Windows 7 x64, and with multiple games including KQ4 and KQ5. }}} {{{#!div style=""font-size: 75%"" Ticket imported from: !#3117434. Ticket imported from: patches/1366. }}}" patch closed normal Engine: SCI MT-32