From 2f1cad0e582f6666fc8ff2b676718bda1be4ac57 Mon Sep 17 00:00:00 2001 From: Vitor Sessak Date: Wed, 6 Aug 2008 05:11:46 +0000 Subject: [PATCH] Simplify rms(): merge a few operations in the same statement Originally committed as revision 14639 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavcodec/ra144.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavcodec/ra144.c b/libavcodec/ra144.c index dab7b1f807..dcb8526268 100644 --- a/libavcodec/ra144.c +++ b/libavcodec/ra144.c @@ -154,7 +154,7 @@ static unsigned int rms(const int *data) { int i; unsigned int res = 0x10000; - int b = 0; + int b = 10; for (i=0; i < 10; i++) { res = (((0x1000000 - data[i]*data[i]) >> 12) * res) >> 12; @@ -168,10 +168,7 @@ static unsigned int rms(const int *data) } } - res = t_sqrt(res); - - res >>= (b + 10); - return res; + return t_sqrt(res) >> b; } static void do_output_subblock(RA144Context *ractx, const uint16_t *lpc_coefs,