har0ke/FFmpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify eval_refl(): do not store a buffer value in a var when the buffer could be used directly

Browse Source 
Originally committed as revision 14652 to svn://svn.ffmpeg.org/ffmpeg/trunk
This commit is contained in:
Vitor Sessak 2008-08-06 21:16:24 +00:00
parent aa425ada54
commit bdb34c9f5e
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
libavcodec/ra144.c
View File

@ -236,15 +236,15 @@ static int eval_refl(int *refl, const int16_t *coefs, RA144Context *ractx)
for (i=0; i < 10; i++) for (i=0; i < 10; i++)
buffer2[i] = coefs[i]; buffer2[i] = coefs[i];
u = refl[9] = bp2[9]; refl[9] = bp2[9];
if (u + 0x1000 > 0x1fff) { if ((unsigned) bp2[9] + 0x1000 > 0x1fff) {
av_log(ractx, AV_LOG_ERROR, "Overflow. Broken sample?\n"); av_log(ractx, AV_LOG_ERROR, "Overflow. Broken sample?\n");
return 1; return 1;
} }
for (c=8; c >= 0; c--) { for (c=8; c >= 0; c--) {
b = 0x1000-((u * u) >> 12); b = 0x1000-((bp2[c+1] * bp2[c+1]) >> 12);
if (!b) if (!b)
b = -2; b = -2;
@ -252,9 +252,9 @@ static int eval_refl(int *refl, const int16_t *coefs, RA144Context *ractx)
for (u=0; u<=c; u++) for (u=0; u<=c; u++)
bp1[u] = ((bp2[u] - ((refl[c+1] * bp2[c-u]) >> 12)) * (0x1000000 / b)) >> 12; bp1[u] = ((bp2[u] - ((refl[c+1] * bp2[c-u]) >> 12)) * (0x1000000 / b)) >> 12;
refl[c] = u = bp1[c]; refl[c] = bp1[c];
if ((u + 0x1000) > 0x1fff) if ((unsigned) bp1[c] + 0x1000 > 0x1fff)
return 1; return 1;
FFSWAP(int *, bp1, bp2); FFSWAP(int *, bp1, bp2);