[FFmpeg-cvslog] swresample/resample: fix rounding errors with filter_size=1 and phase_shift=0

Marton Balint git at videolan.org
Tue Mar 19 22:17:48 EET 2024


ffmpeg | branch: release/5.1 | Marton Balint <cus at passwd.hu> | Tue Feb 27 10:31:31 2024 +0100| [ef327c189f3a4b6e1d3bdbb5269284ff3fa32d03] | committer: Marton Balint

swresample/resample: fix rounding errors with filter_size=1 and phase_shift=0

Depending on input chunk size noticable corrpution was hearable, here is an
example command line:

ffplay -f lavfi -i "sine=440:r=8000:samples_per_frame=32,aresample=24000:filter_size=1:phase_shift=0"

Fix this by rounding the fixed point fractions up instead of down.

Signed-off-by: Marton Balint <cus at passwd.hu>
(cherry picked from commit 7b1b9bb31f04fb9ba06e79c767c1084f0e229bb6)

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ef327c189f3a4b6e1d3bdbb5269284ff3fa32d03
---

 libswresample/resample.c     | 4 ++--
 tests/fate/libswresample.mak | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/libswresample/resample.c b/libswresample/resample.c
index 8f9efc3f21..a91c925574 100644
--- a/libswresample/resample.c
+++ b/libswresample/resample.c
@@ -461,8 +461,8 @@ static int multiple_resample(ResampleContext *c, AudioData *dst, int dst_size, A
     *consumed = 0;
 
     if (c->filter_length == 1 && c->phase_count == 1) {
-        int64_t index2= (1LL<<32)*c->frac/c->src_incr + (1LL<<32)*c->index;
-        int64_t incr= (1LL<<32) * c->dst_incr / c->src_incr;
+        int64_t index2= (1LL<<32)*c->frac/c->src_incr + (1LL<<32)*c->index + 1;
+        int64_t incr= (1LL<<32) * c->dst_incr / c->src_incr + 1;
         int new_size = (src_size * (int64_t)c->src_incr - c->frac + c->dst_incr - 1) / c->dst_incr;
 
         dst_size = FFMAX(FFMIN(dst_size, new_size), 0);
diff --git a/tests/fate/libswresample.mak b/tests/fate/libswresample.mak
index f2108016af..82d5ec4bf9 100644
--- a/tests/fate/libswresample.mak
+++ b/tests/fate/libswresample.mak
@@ -347,13 +347,13 @@ endef
 fate-swr-resample_nn-fltp-44100-8000: CMP_TARGET = 591.03
 fate-swr-resample_nn-fltp-44100-8000: SIZE_TOLERANCE = 529200 - 20486
 
-fate-swr-resample_nn-fltp-8000-44100: CMP_TARGET = 3163.32
+fate-swr-resample_nn-fltp-8000-44100: CMP_TARGET = 3156.61
 fate-swr-resample_nn-fltp-8000-44100: SIZE_TOLERANCE = 96000 - 20480
 
 fate-swr-resample_nn-s16p-44100-8000: CMP_TARGET = 590.98
 fate-swr-resample_nn-s16p-44100-8000: SIZE_TOLERANCE = 529200 - 20486
 
-fate-swr-resample_nn-s16p-8000-44100: CMP_TARGET = 3163.39
+fate-swr-resample_nn-s16p-8000-44100: CMP_TARGET = 3156.61
 fate-swr-resample_nn-s16p-8000-44100: SIZE_TOLERANCE = 96000 - 20480
 
 define ARESAMPLE_ASYNC



More information about the ffmpeg-cvslog mailing list