[Scummvm-cvs-logs] SF.net SVN: scummvm:[35436] scummvm/trunk/engines/saga

[thebluegr at users.sourceforge.net]

Revision: 35436
          http://scummvm.svn.sourceforge.net/scummvm/?rev=35436&view=rev
Author:   thebluegr
Date:     2008-12-19 10:54:05 +0000 (Fri, 19 Dec 2008)

Log Message:
-----------
Fixed bug #2424530 - "IHNM: SFX are no longer played", by assigning correct default values for missing config manager settings

Modified Paths:
--------------
    scummvm/trunk/engines/saga/saga.cpp
    scummvm/trunk/engines/saga/sound.cpp

Modified: scummvm/trunk/engines/saga/saga.cpp
===================================================================
--- scummvm/trunk/engines/saga/saga.cpp	2008-12-19 08:22:25 UTC (rev 35435)
+++ scummvm/trunk/engines/saga/saga.cpp	2008-12-19 10:54:05 UTC (rev 35436)
@@ -141,10 +141,10 @@
 }
 
 Common::Error SagaEngine::init() {
-	_musicVolume = ConfMan.getInt("music_volume");
-	_subtitlesEnabled = ConfMan.getBool("subtitles");
+	_musicVolume = ConfMan.hasKey("music_volume") ? ConfMan.getInt("music_volume") : 255;
+	_subtitlesEnabled = ConfMan.hasKey("subtitles") ? ConfMan.getBool("subtitles") : true;
 	_readingSpeed = getTalkspeed();
-	_copyProtection = ConfMan.getBool("copy_protection");
+	_copyProtection = ConfMan.hasKey("copy_protection") ? ConfMan.getBool("copy_protection") : false;
 	_gf_wyrmkeep = false;
 	_gf_compressed_sounds = false;
 	_musicWasPlaying = false;
@@ -217,7 +217,7 @@
 				_voicesEnabled = true;
 				ConfMan.setBool("voices", true);
 			} else {
-				_voicesEnabled = ConfMan.getBool("voices");
+				_voicesEnabled = ConfMan.hasKey("voices") ? ConfMan.getBool("voices") : true;
 			}
 		} else {
 			_voicesEnabled = true;
@@ -518,17 +518,17 @@
 }
 
 int SagaEngine::getTalkspeed() {
-	return (ConfMan.getInt("talkspeed") * 3 + 255 / 2) / 255;
+	return ((ConfMan.hasKey("talkspeed") ? ConfMan.getInt("talkspeed") : 255) * 3 + 255 / 2) / 255;
 }
 
 void SagaEngine::syncSoundSettings() {
-	_subtitlesEnabled = ConfMan.getBool("subtitles");
+	_subtitlesEnabled = ConfMan.hasKey("subtitles") ? ConfMan.getBool("subtitles") : true;
 	_readingSpeed = getTalkspeed();
 
 	if (_readingSpeed > 3)
 		_readingSpeed = 0;
 
-	_musicVolume = ConfMan.getInt("music_volume");
+	_musicVolume = ConfMan.hasKey("music_volume") ? ConfMan.getInt("music_volume") : 255;
 	_music->setVolume(_musicVolume, 1);
 	_sound->setVolume();
 }

Modified: scummvm/trunk/engines/saga/sound.cpp
===================================================================
--- scummvm/trunk/engines/saga/sound.cpp	2008-12-19 08:22:25 UTC (rev 35435)
+++ scummvm/trunk/engines/saga/sound.cpp	2008-12-19 10:54:05 UTC (rev 35436)
@@ -206,8 +206,8 @@
 }
 
 void Sound::setVolume() {
-	_vm->_soundVolume = ConfMan.getInt("sound_volume");
-	_vm->_speechVolume = ConfMan.getInt("speech_volume");
+	_vm->_soundVolume = ConfMan.hasKey("sound_volume") ? ConfMan.getInt("sound_volume") : 255;
+	_vm->_speechVolume = ConfMan.hasKey("speech_volume") ? ConfMan.getInt("speech_volume") : 255;
 	_mixer->setVolumeForSoundType(Audio::Mixer::kSFXSoundType, _vm->_soundVolume);
 	_mixer->setVolumeForSoundType(Audio::Mixer::kSpeechSoundType, _vm->_speechVolume);
 }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.