duuqnd/cave-story-solaris
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Correct Lanczos array initialisation

Browse source 
I must have forgotten that Lanczos reads backwards too
This commit is contained in:
Clownacy 2020-09-18 00:59:05 +01:00
parent d3e18d795d
commit 4763aecdbd
1 changed files with 12 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/Backends/Audio/SoftwareMixer/Mixer.cpp
View file

@ -67,10 +67,6 @@ Mixer_Sound* Mixer_CreateSound(unsigned int frequency, const unsigned char *samp
} }
#ifdef LANCZOS_RESAMPLER #ifdef LANCZOS_RESAMPLER
// Blank samples outside the array bounds (we'll deal with the other half later)
for (size_t i = 0; i < LANCZOS_KERNEL_RADIUS - 1; ++i)
sound->samples[i] = 0;
sound->samples += LANCZOS_KERNEL_RADIUS - 1; sound->samples += LANCZOS_KERNEL_RADIUS - 1;
#endif #endif
@ -118,11 +114,21 @@ void Mixer_PlaySound(Mixer_Sound *sound, bool looping)
// either blank samples or repeated samples // either blank samples or repeated samples
#ifdef LANCZOS_RESAMPLER #ifdef LANCZOS_RESAMPLER
if (looping) if (looping)
for (size_t i = 0; i < LANCZOS_KERNEL_RADIUS; ++i) {
for (int i = -LANCZOS_KERNEL_RADIUS + 1; i < 0; ++i)
sound->samples[i] = sound->samples[sound->frames + i];
for (int i = 0; i < LANCZOS_KERNEL_RADIUS; ++i)
sound->samples[sound->frames + i] = sound->samples[i]; sound->samples[sound->frames + i] = sound->samples[i];
}
else else
for (size_t i = 0; i < LANCZOS_KERNEL_RADIUS; ++i) {
for (int i = -LANCZOS_KERNEL_RADIUS + 1; i < 0; ++i)
sound->samples[i] = 0;
for (int i = 0; i < LANCZOS_KERNEL_RADIUS; ++i)
sound->samples[sound->frames + i] = 0; sound->samples[sound->frames + i] = 0;
}
#else #else
sound->samples[sound->frames] = looping ? sound->samples[0] : 0; sound->samples[sound->frames] = looping ? sound->samples[0] : 0;
#endif #endif