From: Michael Niedermayer Date: Wed, 31 Jan 2007 10:07:22 +0000 (+0000) Subject: replace if(x>>b) by if(x>C) as shifts are slow on some cpus and i have my doubts... X-Git-Tag: v0.5~10281 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=801778bc8306e4b58d8c27022aa7903e86a2b040;p=platform%2Fupstream%2Flibav.git replace if(x>>b) by if(x>C) as shifts are slow on some cpus and i have my doubts that gcc can replace the shifts as x is signed, it could in theory but well its gcc ... Originally committed as revision 7776 to svn://svn.ffmpeg.org/ffmpeg/trunk --- diff --git a/libavcodec/lzo.c b/libavcodec/lzo.c index e4a1111..54a34c2 100644 --- a/libavcodec/lzo.c +++ b/libavcodec/lzo.c @@ -184,11 +184,11 @@ int lzo1x_decode(void *out, int *outlen, void *in, int *inlen) { } while (!c.error) { int cnt, back; - if (x >> 4) { - if (x >> 6) { + if (x > 15) { + if (x > 63) { cnt = (x >> 5) - 1; back = (GETB(c) << 3) + ((x >> 2) & 7) + 1; - } else if (x >> 5) { + } else if (x > 31) { cnt = get_len(&c, x, 31); x = GETB(c); back = (GETB(c) << 6) + (x >> 2) + 1;