Simplify rms(): merge a few operations in the same statement
authorVitor Sessak <vitor1001@gmail.com>
Wed, 6 Aug 2008 05:11:46 +0000 (05:11 +0000)
committerVitor Sessak <vitor1001@gmail.com>
Wed, 6 Aug 2008 05:11:46 +0000 (05:11 +0000)
Originally committed as revision 14639 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/ra144.c

index dab7b1f80701ac13cf939952d31e1eaf37e19434..dcb852626892ef46420176bcc10ca371ae29eb1f 100644 (file)
@@ -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,