Fix issues with speech

Author: IonAgorria

Source/Game/Runtime.cpp

@@ -832,7 +832,7 @@ void FinitSound()
 	IniManager ini("Perimeter.ini");
 	ini.putFloat("Sound","SoundVolume", terSoundVolume);
     ini.putFloat("Sound","MusicVolume", terMusicVolume);
-    ini.putFloat("Sound","SpeechVolume", terVoiceVolume);
+    ini.putFloat("Sound","SpeechVolume", terSpeechVolume);
     ini.putFloat("Sound","VoiceVolume", terVoiceVolume);
 
 	SNDReleaseSound();

Source/Sound/AudioPlayer.cpp

@@ -34,7 +34,7 @@ bool SpeechPlayer::OpenToPlay(const char* fname, bool cycled) {
     sample = SNDLoadSound(fname);
     if (!sample) return false;
     sample->looped = cycled; //Tecnically not need for speeches but whatever
-    sample->channel_group = SND_GROUP_SPEECH;
+    sample->channel_group = channel_group;
     sample->global_volume_select = global_volume_select;
     sample->volume = volume;
     sample->steal_channel = true; //Just in case another speech is playing

Source/Sound/AudioPlayer.h

@@ -40,6 +40,8 @@ class SpeechPlayer: public AudioPlayer {
     void destroySample();
 
 public:
+    int channel_group = SND_GROUP_SPEECH;
+    
     SpeechPlayer();
     ~SpeechPlayer() override;
 

Source/Sound/Sample.h

@@ -1,14 +1,6 @@
 #ifndef PERIMETER_SAMPLE_H
 #define PERIMETER_SAMPLE_H
 
-//Mixer channel groups
-#define SND_GROUP_SPEECH 0
-#define SND_GROUP_EFFECTS 1
-#define SND_GROUP_EFFECTS_ONCE 2
-#define SND_GROUP_EFFECTS_LOOPED 3
-
-#define SND_NO_CHANNEL -1
-
 #include "SampleParams.h"
 
 //Wrapper for chunk that will be freed once unused

Source/Sound/SampleParams.h

@@ -4,6 +4,14 @@
 #ifndef PERIMETER_SAMPLEPARAMS_H
 #define PERIMETER_SAMPLEPARAMS_H
 
+//Mixer channel groups
+#define SND_GROUP_SPEECH 0
+#define SND_GROUP_EFFECTS 1
+#define SND_GROUP_EFFECTS_ONCE 2
+#define SND_GROUP_EFFECTS_LOOPED 3
+
+#define SND_NO_CHANNEL -1
+
 //Which volume to use
 enum GLOBAL_VOLUME {
     GLOBAL_VOLUME_CHANNEL = 0, //Use voice or effects volume according to current channel

Source/UserInterface/HistoryScene.cpp

@@ -541,12 +541,6 @@ void HistoryScene::postDraw() {
 	scene->PostDraw(historyCamera->getCamera());
 }
 
-void HistoryScene::setupAudio() {
-	if (terSpeechVolume == 0) {
-		stopAudio();
-	}
-}
-
 void HistoryScene::startAudio(const string& name) {
 	if (!name.empty()) {
 		stopAudio();

Source/UserInterface/HistoryScene.h

@@ -16,7 +16,6 @@ class HistoryScene : public Commander {
 
 		void init(cVisGeneric* visGeneric, bool bw, bool addBlendAlpha = true);
 		void done();
-		void setupAudio();
 		void quant(const Vect2f& mousePos, float dt);
 		void preDraw();
 		void draw();

Source/UserInterface/OptionsMenu.cpp

@@ -464,10 +464,15 @@ void OnComboGraphicsMode(CShellWindow* pWnd, InterfaceEventCode code, int param)
 }
 
 // sound
-static SpeechPlayer* OptionSamplePlayer = new SpeechPlayer();
+static SpeechPlayer* OptionSamplePlayer = nullptr;
 
 bool OptionPlaySample(GLOBAL_VOLUME global_volume, float volume, const char* path) {
     bool started = false;
+    if (!OptionSamplePlayer) {
+        OptionSamplePlayer = new SpeechPlayer();
+        //Avoid overlapping any running speech audios
+        OptionSamplePlayer->channel_group = SND_GROUP_EFFECTS;
+    }
     if (OptionSamplePlayer->GetVolumeSelection() != global_volume) {
         OptionSamplePlayer->Stop();
     }
@@ -502,16 +507,14 @@ void OnSliderSpeechVolume(CShellWindow* pWnd, InterfaceEventCode code, int param
     CSliderWindow *pSlider = (CSliderWindow*) pWnd;
     if ( code == EVENT_CREATEWND ) {
         pSlider->pos = terSpeechVolume;
-        historyScene.setupAudio();
-        _shellIconManager.setupAudio();
     } else if((code == EVENT_SLIDERUPDATE && pSlider->pos != terSpeechVolume) || code == EVENT_UNPRESSED) {
         terSpeechVolume = pSlider->pos;
         if (code == EVENT_UNPRESSED) {
-            historyScene.setupAudio();
             _shellIconManager.setupAudio();
         }
         std::string path = getLocDataPath();
         static int i = 0;
+        //Avoid using briefing audios in GW since different languages may have different filenames
         //Select ET audio if only ET is selected
         bool et = terGameContentSelect == PERIMETER_ET;
         if (1 < i) i = 0;

Source/UserInterface/PerimeterShellDisp.cpp

@@ -912,7 +912,7 @@ void CShellIconManager::Done()
 		SNDEnableVoices(0 < terVoiceVolume);
 		speechSound->Stop();
 		delete speechSound;
-		speechSound = 0;
+		speechSound = nullptr;
 	}
 
 	cutSceneModeOn = false;
@@ -1462,12 +1462,9 @@ void CShellIconManager::playGameOverSound(const char* path) {
 }
 
 void CShellIconManager::setupAudio() {
-	if (speechSound) {
-		if (terSpeechVolume == 0) {
-			speechSound->Stop();
-		}
-        speechSound->SetVolumeSelection(GLOBAL_VOLUME_IGNORE); //We set volume here manually
-		speechSound->SetVolume(terVoiceVolume);
+	if (speechSound && speechSound->IsPlay()) {
+        //Set volume, a speech might be playing while user is adjusting
+		speechSound->SetVolume(terSpeechVolume);
 	}
 }