fix: Stability fixes.

AmplitudeAudio · Dec 5, 2024 · 239c7d3 · 239c7d3
1 parent 1f41bc7
commit 239c7d3
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/Core/Playback/ChannelInternalState.cpp b/src/Core/Playback/ChannelInternalState.cpp
@@ -45,6 +45,8 @@ namespace SparkyStudios::Audio::Amplitude
         priority_node.remove();
         bus_node.remove();
         entity_node.remove();
+        room_node.remove();
+        listener_node.remove();
     }
 
     void ChannelInternalState::Reset()

diff --git a/src/Core/RoomInternalState.cpp b/src/Core/RoomInternalState.cpp
@@ -48,9 +48,10 @@ namespace SparkyStudios::Audio::Amplitude
 
     RoomMaterial::RoomMaterial(RoomMaterialType type)
         : m_type(type)
+        , m_absorptionCoefficients{ 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f }
     {
         const auto& coefficients = kDefaultAbsorptionCoefficients[static_cast<AmUInt32>(type)];
-        std::copy(std::begin(coefficients), std::begin(coefficients) + 9, m_absorptionCoefficients);
+        std::copy_n(std::begin(coefficients), 9, m_absorptionCoefficients);
     }
 
     void RoomInternalState::Update()
@@ -66,7 +67,7 @@ namespace SparkyStudios::Audio::Amplitude
             const auto& coefficients = _materials[i].m_absorptionCoefficients;
 
             // Compute average absorption coefficients
-            AmReal32 averageAbsorptionCoefficients =
+            const AmReal32 averageAbsorptionCoefficients =
                 std::accumulate(
                     std::begin(coefficients) + kReflectionStartingBand,
                     std::begin(coefficients) + kReflectionStartingBand + kReflectionAveragingBandsCount, 0.0f) /

diff --git a/src/Sound/Schedulers/RandomScheduler.cpp b/src/Sound/Schedulers/RandomScheduler.cpp
@@ -57,7 +57,7 @@ namespace SparkyStudios::Audio::Amplitude
     Sound* RandomScheduler::Select(const std::vector<AmSoundID>& toSkip)
     {
     Pick:
-        float selection = std::rand() / static_cast<float>(RAND_MAX) * _probabilitiesSum;
+        AmReal32 selection = static_cast<AmReal32>(std::rand()) / static_cast<AmReal32>(RAND_MAX) * _probabilitiesSum;
         for (flatbuffers::uoffset_t i = 0; i < _sounds.size(); ++i)
         {
             const auto* entry = _definition->sounds()->GetAs<RandomSchedulerCollectionEntry>(i);