From 40812fdcf1dc74c026c93cc5168a2b10a2b0c777 Mon Sep 17 00:00:00 2001 From: Vitor Sessak Date: Wed, 6 Aug 2008 21:02:56 +0000 Subject: 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 --- libavcodec/ra144.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'libavcodec/ra144.c') 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; -- cgit v1.2.3