Mercurial > libavcodec.hg
changeset 7515:1d5800cec7ad libavcodec
Simplify eval_refl(): do not store a buffer value in a var when the buffer could be used directly
author | vitor |
---|---|
date | Wed, 06 Aug 2008 21:16:24 +0000 |
parents | e0e075680996 |
children | 11fe8c09948e |
files | ra144.c |
diffstat | 1 files changed, 5 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/ra144.c Wed Aug 06 21:04:40 2008 +0000 +++ b/ra144.c Wed Aug 06 21:16:24 2008 +0000 @@ -236,15 +236,15 @@ for (i=0; i < 10; 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"); return 1; } for (c=8; c >= 0; c--) { - b = 0x1000-((u * u) >> 12); + b = 0x1000-((bp2[c+1] * bp2[c+1]) >> 12); if (!b) b = -2; @@ -252,9 +252,9 @@ for (u=0; u<=c; u++) 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; FFSWAP(int *, bp1, bp2);