[FFmpeg-devel] [PATCH 24/25] avfilter/af_headphone: Remove pointless additions

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Wed Sep 9 00:18:55 EEST 2020


buffer_length is a power-of-two and modulo is buffer_length - 1, so that
buffer_length & modulo is zero.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
 libavfilter/af_headphone.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/af_headphone.c b/libavfilter/af_headphone.c
index 8db546adbf..03ef88c4a4 100644
--- a/libavfilter/af_headphone.c
+++ b/libavfilter/af_headphone.c
@@ -178,7 +178,7 @@ static int headphone_convolute(AVFilterContext *ctx, void *arg, int jobnr, int n
                 continue;
             }
 
-            read = (wr - (ir_len - 1) + buffer_length) & modulo;
+            read = (wr - (ir_len - 1)) & modulo;
 
             if (read + ir_len < buffer_length) {
                 memcpy(temp_src, bptr + read, ir_len * sizeof(*temp_src));
-- 
2.20.1



More information about the ffmpeg-devel mailing list