Fixed up some dumdum issues with SelectNextSounds, and updated method comments

Author: garethyr

Entities/SoundSet.cpp

@@ -224,6 +224,11 @@ namespace RTE {
 
 	bool SoundSet::SelectNextSounds() {
 		if (m_SoundSelectionCycleMode == SoundSelectionCycleMode::ALL) {
+			for (SoundSet &subSoundSet : m_SubSoundSets) {
+				if (!subSoundSet.SelectNextSounds()) {
+					return false;
+				}
+			}
 			return true;
 		}
 		int selectedVectorSize = m_CurrentSelection.first == false ? m_SoundData.size() : m_SubSoundSets.size();
@@ -265,7 +270,6 @@ namespace RTE {
 				return false;
 			case 1:
 				if (selectedVectorSize == 0) { m_CurrentSelection.first = !m_CurrentSelection.first; }
-				return true;
 			case 2:
 				selectSoundForwards();
 				break;
@@ -283,6 +287,9 @@ namespace RTE {
 				}
 				RTEAssert(m_CurrentSelection.second >= 0 && m_CurrentSelection.second < selectedVectorSize, "Failed to select next sound, either none was selected or the selected sound was invalid.");
 		}
+		if (m_CurrentSelection.first == true) {
+			return m_SubSoundSets[m_CurrentSelection.second].SelectNextSounds();
+		}
 		return true;
 	}
 }

Entities/SoundSet.h

@@ -175,9 +175,9 @@ namespace RTE {
 
 #pragma region Miscellaneous
 		/// <summary>
-		/// Selects the next sounds of this SoundContainer to be played.
+		/// Selects the next sounds of this SoundSet to be played, also selecting them for sub SoundSets as appropriate.
 		/// </summary>
-		/// <returns>Whether a new sound set was successfully selected.</returns>
+		/// <returns>False if this SoundSet or any of its sub SoundSets failed to select sounds, or true if everything worked.</returns>
 		bool SelectNextSounds();
 #pragma endregion