mirror/RetroArch
1
0
Fork 0
mirror of https://github.com/libretro/RetroArch synced 2025-02-06 09:40:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

(Audio Mixer) Increase sample buffer padding (#13300)

Browse Source
This commit is contained in:
jdgleaver 2021-11-30 13:52:25 +00:00 committed by GitHub
parent a585e05a8a
commit 67eb42a8c2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
libretro-common/audio/audio_mixer.c
View File

@ -280,14 +280,14 @@ static bool one_shot_resample(const float* in, size_t samples_in,
return false; return false;
/* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We /* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We
* add four more samples in the formula below just as safeguard, because * add 16 more samples in the formula below just as safeguard, because
* resampler->process sometimes reports more output samples than the * resampler->process sometimes reports more output samples than the
* formula below calculates. Ideally, audio resamplers should have a * formula below calculates. Ideally, audio resamplers should have a
* function to return the number of samples they will output given a * function to return the number of samples they will output given a
* count of input samples. */ * count of input samples. */
*samples_out = (size_t)(samples_in * ratio); *samples_out = (size_t)(samples_in * ratio);
*out = (float*)memalign_alloc(16, *out = (float*)memalign_alloc(16,
(((*samples_out + 4) + 15) & ~15) * sizeof(float)); (((*samples_out + 16) + 15) & ~15) * sizeof(float));
if (*out == NULL) if (*out == NULL)
return false; return false;
@ -546,14 +546,14 @@ static bool audio_mixer_play_ogg(
} }
/* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We /* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We
* add four more samples in the formula below just as safeguard, because * add 16 more samples in the formula below just as safeguard, because
* resampler->process sometimes reports more output samples than the * resampler->process sometimes reports more output samples than the
* formula below calculates. Ideally, audio resamplers should have a * formula below calculates. Ideally, audio resamplers should have a
* function to return the number of samples they will output given a * function to return the number of samples they will output given a
* count of input samples. */ * count of input samples. */
samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio); samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio);
ogg_buffer = (float*)memalign_alloc(16, ogg_buffer = (float*)memalign_alloc(16,
(((samples + 4) + 15) & ~15) * sizeof(float)); (((samples + 16) + 15) & ~15) * sizeof(float));
if (!ogg_buffer) if (!ogg_buffer)
{ {
@ -695,14 +695,14 @@ static bool audio_mixer_play_flac(
} }
/* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We /* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We
* add four more samples in the formula below just as safeguard, because * add 16 more samples in the formula below just as safeguard, because
* resampler->process sometimes reports more output samples than the * resampler->process sometimes reports more output samples than the
* formula below calculates. Ideally, audio resamplers should have a * formula below calculates. Ideally, audio resamplers should have a
* function to return the number of samples they will output given a * function to return the number of samples they will output given a
* count of input samples. */ * count of input samples. */
samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio); samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio);
flac_buffer = (float*)memalign_alloc(16, flac_buffer = (float*)memalign_alloc(16,
(((samples + 4) + 15) & ~15) * sizeof(float)); (((samples + 16) + 15) & ~15) * sizeof(float));
if (!flac_buffer) if (!flac_buffer)
{ {
@ -773,14 +773,14 @@ static bool audio_mixer_play_mp3(
} }
/* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We /* Allocate on a 16-byte boundary, and pad to a multiple of 16 bytes. We
* add four more samples in the formula below just as safeguard, because * add 16 more samples in the formula below just as safeguard, because
* resampler->process sometimes reports more output samples than the * resampler->process sometimes reports more output samples than the
* formula below calculates. Ideally, audio resamplers should have a * formula below calculates. Ideally, audio resamplers should have a
* function to return the number of samples they will output given a * function to return the number of samples they will output given a
* count of input samples. */ * count of input samples. */
samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio); samples = (unsigned)(AUDIO_MIXER_TEMP_BUFFER * ratio);
mp3_buffer = (float*)memalign_alloc(16, mp3_buffer = (float*)memalign_alloc(16,
(((samples + 4) + 15) & ~15) * sizeof(float)); (((samples + 16) + 15) & ~15) * sizeof(float));
if (!mp3_buffer) if (!mp3_buffer)
{ {