From 40f59cacda872a494cf033a6f74da7feae282367 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Thu, 9 Oct 2008 15:18:09 +0200 Subject: pcm: fixed software volume, broken by unsigned integer "volume" was passed as an unsigned integer, which is correct. It's just that when it was multiplied with the sample value, the whole operation was changed to unsigned, breaking the algorithm (and Qball's ears). Internally change "volume" to signed. --- src/pcm_utils.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'src/pcm_utils.c') diff --git a/src/pcm_utils.c b/src/pcm_utils.c index 17449f7d2..c3526f9d1 100644 --- a/src/pcm_utils.c +++ b/src/pcm_utils.c @@ -47,8 +47,7 @@ pcm_range(int32_t sample, unsigned bits) } static void -pcm_volume_change_8(int8_t *buffer, unsigned num_samples, - unsigned volume) +pcm_volume_change_8(int8_t *buffer, unsigned num_samples, int volume) { while (num_samples > 0) { int32_t sample = *buffer; @@ -61,8 +60,7 @@ pcm_volume_change_8(int8_t *buffer, unsigned num_samples, } static void -pcm_volume_change_16(int16_t *buffer, unsigned num_samples, - unsigned volume) +pcm_volume_change_16(int16_t *buffer, unsigned num_samples, int volume) { while (num_samples > 0) { int32_t sample = *buffer; @@ -75,8 +73,7 @@ pcm_volume_change_16(int16_t *buffer, unsigned num_samples, } static void -pcm_volume_change_24(int32_t *buffer, unsigned num_samples, - unsigned volume) +pcm_volume_change_24(int32_t *buffer, unsigned num_samples, int volume) { while (num_samples > 0) { int64_t sample = *buffer; -- cgit v1.2.3