diff options
author | bunnei <bunneidev@gmail.com> | 2021-07-13 07:09:05 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-07-13 07:09:05 +0200 |
commit | 7d464f73c9990329d14617de71653612de46085b (patch) | |
tree | 0cdb2d11e9bf0a7084aa4a5f8421d907291e1053 | |
parent | Merge pull request #6597 from FernandoS27/accelerate-dma (diff) | |
parent | Replace NaN mix volume samples with silence. (diff) | |
download | yuzu-7d464f73c9990329d14617de71653612de46085b.tar yuzu-7d464f73c9990329d14617de71653612de46085b.tar.gz yuzu-7d464f73c9990329d14617de71653612de46085b.tar.bz2 yuzu-7d464f73c9990329d14617de71653612de46085b.tar.lz yuzu-7d464f73c9990329d14617de71653612de46085b.tar.xz yuzu-7d464f73c9990329d14617de71653612de46085b.tar.zst yuzu-7d464f73c9990329d14617de71653612de46085b.zip |
-rw-r--r-- | src/audio_core/command_generator.cpp | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/audio_core/command_generator.cpp b/src/audio_core/command_generator.cpp index b99d0fc91..45b2eef52 100644 --- a/src/audio_core/command_generator.cpp +++ b/src/audio_core/command_generator.cpp @@ -42,6 +42,15 @@ void ApplyMix(std::span<s32> output, std::span<const s32> input, s32 gain, s32 s s32 ApplyMixRamp(std::span<s32> output, std::span<const s32> input, float gain, float delta, s32 sample_count) { + // XC2 passes in NaN mix volumes, causing further issues as we handle everything as s32 rather + // than float, so the NaN propogation is lost. As the samples get further modified for + // volume etc, they can get out of NaN range, so a later heuristic for catching this is + // more difficult. Handle it here by setting these samples to silence. + if (std::isnan(gain)) { + gain = 0.0f; + delta = 0.0f; + } + s32 x = 0; for (s32 i = 0; i < sample_count; i++) { x = static_cast<s32>(static_cast<float>(input[i]) * gain); |