Browse code
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
Vitor Sessak authored on 2008/08/07 06:02:56Showing 1 changed files
| ... | ... |
@@ -244,16 +244,10 @@ static int eval_refl(int *refl, const int16_t *coefs, RA144Context *ractx) |
| 244 | 244 |
} |
| 245 | 245 |
|
| 246 | 246 |
for (c=8; c >= 0; c--) {
|
| 247 |
- if (u == 0x1000) |
|
| 248 |
- u++; |
|
| 249 |
- |
|
| 250 |
- if (u == 0xfffff000) |
|
| 251 |
- u--; |
|
| 252 |
- |
|
| 253 | 247 |
b = 0x1000-((u * u) >> 12); |
| 254 | 248 |
|
| 255 | 249 |
if (b == 0) |
| 256 |
- b++; |
|
| 250 |
+ b = -2; |
|
| 257 | 251 |
|
| 258 | 252 |
for (u=0; u<=c; u++) |
| 259 | 253 |
bp1[u] = ((bp2[u] - ((refl[c+1] * bp2[c-u]) >> 12)) * (0x1000000 / b)) >> 12; |