From 956b69b655026c6c84bd4f3ab5b9b97dc91fea8f Mon Sep 17 00:00:00 2001 From: Johannes Schickel Date: Wed, 26 Dec 2007 01:17:58 +0000 Subject: [PATCH] Update comment. svn-id: r29998 --- engines/kyra/kyra.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/engines/kyra/kyra.cpp b/engines/kyra/kyra.cpp index e1ead2a873a..f296eade709 100644 --- a/engines/kyra/kyra.cpp +++ b/engines/kyra/kyra.cpp @@ -89,7 +89,10 @@ int KyraEngine::init() { _mixer->setVolumeForSoundType(Audio::Mixer::kMusicSoundType, ConfMan.getInt("music_volume")); _mixer->setVolumeForSoundType(Audio::Mixer::kSpeechSoundType, ConfMan.getInt("speech_volume")); - // for now we prefer Adlib over native MIDI + // We prefer AdLib over native MIDI, since our AdLib playback code is much + // more mature than our MIDI player. For example we are missing MT-32 support + // and it seems our MIDI playback code has threading issues (see bug #1506583 + // "KYRA1: Crash on exceeded polyphony" for more information). int midiDriver = MidiDriver::detectMusicDriver(MDT_MIDI | MDT_ADLIB/* | MDT_PREFER_MIDI*/); if (_flags.platform == Common::kPlatformFMTowns || _flags.platform == Common::kPlatformPC98) { @@ -119,8 +122,7 @@ int KyraEngine::init() { // Unlike some SCUMM games, it's not that the MIDI sounds are // missing. It's just that at least at the time of writing they // are decidedly inferior to the Adlib ones. - - if (midiDriver != MD_ADLIB && ConfMan.getBool("multi_midi")) { + if (ConfMan.getBool("multi_midi")) { SoundAdlibPC *adlib = new SoundAdlibPC(this, _mixer); assert(adlib);