Simplify range checking in eval_refl(). Before the

condition b == 0 could never be true, because it would
have already been caught in the previous ifs. Also the
condition b == 0x1000 could never be true (it would
triggered the return).

Originally committed as revision 14650 to svn://svn.ffmpeg.org/ffmpeg/trunk

1 files changed, 1 insertions, 7 deletions

diff --git a/libavcodec/ra144.c b/libavcodec/ra144.c
index 6a8349492b..116b39b8da 100644
--- a/libavcodec/ra144.c
+++ b/libavcodec/ra144.c
@@ -244,16 +244,10 @@ static int eval_refl(int *refl, const int16_t *coefs, RA144Context *ractx)
     }
 
     for (c=8; c >= 0; c--) {
-        if (u == 0x1000)
-            u++;
-
-        if (u == 0xfffff000)
-            u--;
-
         b = 0x1000-((u * u) >> 12);
 
         if (b == 0)
-            b++;
+            b = -2;
 
         for (u=0; u<=c; u++)
             bp1[u] = ((bp2[u] - ((refl[c+1] * bp2[c-u]) >> 12)) * (0x1000000 / b)) >> 12;