[FFmpeg-cvslog] ape: Unbreak adaptcoeffs computation
Luca Barbato
git at videolan.org
Thu May 12 15:32:36 CEST 2016
ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Thu Apr 21 16:09:38 2016 +0200| [dd4fb2339f76a958bd7e63e5ac18b8c10852ae1a] | committer: Derek Buitenhuis
ape: Unbreak adaptcoeffs computation
And simplify and explain the expression.
Fault introduced in f3fdef108eb06b1e71b29152bf6822519e787efe
Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=dd4fb2339f76a958bd7e63e5ac18b8c10852ae1a
---
libavcodec/apedec.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c
index b028e94..1e89a7b 100644
--- a/libavcodec/apedec.c
+++ b/libavcodec/apedec.c
@@ -1284,8 +1284,16 @@ static void do_apply_filter(APEContext *ctx, int version, APEFilter *f,
/* Update the adaption coefficients */
absres = FFABS(res);
if (absres)
- *f->adaptcoeffs = ((res & INT32_MIN) ^ ((~0UL) << 30)) >>
- (25 + (absres <= f->avg*3) + (absres <= f->avg*4/3));
+ *f->adaptcoeffs = APESIGN(res) *
+ (8 << ((absres > f->avg * 3) + (absres > f->avg * 4 / 3)));
+ /* equivalent to the following code
+ if (absres <= f->avg * 4 / 3)
+ *f->adaptcoeffs = APESIGN(res) * 8;
+ else if (absres <= f->avg * 3)
+ *f->adaptcoeffs = APESIGN(res) * 16;
+ else
+ *f->adaptcoeffs = APESIGN(res) * 32;
+ */
else
*f->adaptcoeffs = 0;
More information about the ffmpeg-cvslog
mailing list