Made crash in DynamicSongSection at least assert first

Causeless · web-flow · commit 8993e9dd2fd6 · 2024-12-19T12:51:39.000Z
diff --git a/Source/Entities/DynamicSong.cpp b/Source/Entities/DynamicSong.cpp
@@ -162,12 +162,13 @@ SoundContainer& DynamicSongSection::SelectTransitionSoundContainer() {
 		}
 	}
 
+	RTEAssert(!m_TransitionSoundContainers.empty(), "Tried to get a transition SoundContainer from a DynamicSongSection with none to choose from!");
 	return m_TransitionSoundContainers[0];
 }
 
 SoundContainer& DynamicSongSection::SelectSoundContainer() {
 	// Shuffle between our options if we have multiple
-	if (m_SoundContainers.size() != 1) {
+	if (m_SoundContainers.size() > 1) {
 		if (m_ShuffleUnplayedIndices.empty()) {
 			for (unsigned int i = 0; i < m_SoundContainers.size(); i++) {
 				if (i != m_LastSoundContainerIndex) {

Original file line number	Diff line number	Diff line change
`@@ -162,12 +162,13 @@ SoundContainer& DynamicSongSection::SelectTransitionSoundContainer() {`
`162`	`162`	`}`
`163`	`163`	`}`
`164`	`164`
	`165`	`+ RTEAssert(!m_TransitionSoundContainers.empty(), "Tried to get a transition SoundContainer from a DynamicSongSection with none to choose from!");`
`165`	`166`	`return m_TransitionSoundContainers[0];`
`166`	`167`	`}`
`167`	`168`
`168`	`169`	`SoundContainer& DynamicSongSection::SelectSoundContainer() {`
`169`	`170`	`// Shuffle between our options if we have multiple`
`170`		`- if (m_SoundContainers.size() != 1) {`
	`171`	`+ if (m_SoundContainers.size() > 1) {`
`171`	`172`	`if (m_ShuffleUnplayedIndices.empty()) {`
`172`	`173`	`for (unsigned int i = 0; i < m_SoundContainers.size(); i++) {`
`173`	`174`	`if (i != m_LastSoundContainerIndex) {`