From 82d72ef39fd2b06b522ca64905f2de3ad5b462a5 Mon Sep 17 00:00:00 2001 From: Uoti Urpala Date: Sat, 12 Jan 2013 15:19:07 +0200 Subject: mixer: keep fractional part of volume setting mixer_setvolume() accepts float values for volume, but used the integer function av_clip() to limit range, losing the fractional part as a side effect. Change the code to use av_clipf() instead. For most uses this shouldn't make any real difference; actual AO volume settings may not have that much precision anyway. --- audio/mixer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/audio/mixer.c b/audio/mixer.c index eb756f90b4..bdcb0cbc5c 100644 --- a/audio/mixer.c +++ b/audio/mixer.c @@ -124,8 +124,8 @@ static void setvolume_internal(mixer_t *mixer, float l, float r) void mixer_setvolume(mixer_t *mixer, float l, float r) { checkvolume(mixer); // to check mute status and AO support for volume - mixer->vol_l = av_clip(l, 0, 100); - mixer->vol_r = av_clip(r, 0, 100); + mixer->vol_l = av_clipf(l, 0, 100); + mixer->vol_r = av_clipf(r, 0, 100); if (!mixer->ao || mixer->muted_using_volume) return; setvolume_internal(mixer, mixer->vol_l, mixer->vol_r); -- cgit v1.2.3