From: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Date: Fri, 28 Aug 2020 13:28:33 +0000 (+0200)
Subject: avfilter/af_headphone: Remove pointless additions
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=7b841cf6b706055164d5db3f2b901ed32e8092af;p=ffmpeg

avfilter/af_headphone: Remove pointless additions

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

Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---

diff --git a/libavfilter/af_headphone.c b/libavfilter/af_headphone.c
index 8db546adbfc..03ef88c4a42 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));