Skip to content

Commit 51a3494

Browse files
authored
refactor(audio): Replace some index values with enum values in AudioManager (#1364)
1 parent 3574fdd commit 51a3494

File tree

2 files changed

+17
-11
lines changed

2 files changed

+17
-11
lines changed

Core/GameEngine/Include/Common/GameAudio.h

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,14 @@ class AudioManager : public SubsystemInterface
355355

356356
AudioEventRTS *m_silentAudioEvent;
357357

358-
enum { NUM_VOLUME_TYPES = 4 };
358+
enum
359+
{
360+
VOLUME_TYPE_MUSIC,
361+
VOLUME_TYPE_SOUND,
362+
VOLUME_TYPE_SOUND3D,
363+
VOLUME_TYPE_SPEECH,
364+
NUM_VOLUME_TYPES
365+
};
359366
Real *m_savedValues;
360367

361368
// Group of 8

Core/GameEngine/Source/Common/Audio/GameAudio.cpp

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1103,10 +1103,10 @@ void AudioManager::loseFocus( void )
11031103

11041104
// In this case, make all the audio go silent.
11051105
m_savedValues = NEW Real[NUM_VOLUME_TYPES];
1106-
m_savedValues[0] = m_systemMusicVolume;
1107-
m_savedValues[1] = m_systemSoundVolume;
1108-
m_savedValues[2] = m_systemSound3DVolume;
1109-
m_savedValues[3] = m_systemSpeechVolume;
1106+
m_savedValues[VOLUME_TYPE_MUSIC] = m_systemMusicVolume;
1107+
m_savedValues[VOLUME_TYPE_SOUND] = m_systemSoundVolume;
1108+
m_savedValues[VOLUME_TYPE_SOUND3D] = m_systemSound3DVolume;
1109+
m_savedValues[VOLUME_TYPE_SPEECH] = m_systemSpeechVolume;
11101110

11111111
// Now, set them all to 0.
11121112
setVolume(0.0f, (AudioAffect) (AudioAffect_All | AudioAffect_SystemSetting));
@@ -1115,15 +1115,14 @@ void AudioManager::loseFocus( void )
11151115
//-------------------------------------------------------------------------------------------------
11161116
void AudioManager::regainFocus( void )
11171117
{
1118-
if (!m_savedValues) {
1118+
if (!m_savedValues)
11191119
return;
1120-
}
11211120

11221121
// We got focus back. Restore the previous audio values.
1123-
setVolume(m_savedValues[0], (AudioAffect) (AudioAffect_Music | AudioAffect_SystemSetting));
1124-
setVolume(m_savedValues[1], (AudioAffect) (AudioAffect_Sound | AudioAffect_SystemSetting));
1125-
setVolume(m_savedValues[2], (AudioAffect) (AudioAffect_Sound3D | AudioAffect_SystemSetting));
1126-
setVolume(m_savedValues[3], (AudioAffect) (AudioAffect_Speech | AudioAffect_SystemSetting));
1122+
setVolume(m_savedValues[VOLUME_TYPE_MUSIC], (AudioAffect) (AudioAffect_Music | AudioAffect_SystemSetting));
1123+
setVolume(m_savedValues[VOLUME_TYPE_SOUND], (AudioAffect) (AudioAffect_Sound | AudioAffect_SystemSetting));
1124+
setVolume(m_savedValues[VOLUME_TYPE_SOUND3D], (AudioAffect) (AudioAffect_Sound3D | AudioAffect_SystemSetting));
1125+
setVolume(m_savedValues[VOLUME_TYPE_SPEECH], (AudioAffect) (AudioAffect_Speech | AudioAffect_SystemSetting));
11271126

11281127
// Now, blow away the old volumes.
11291128
delete [] m_savedValues;

0 commit comments

Comments
 (0)