[Ffmpeg-cvslog] r7776 - trunk/libavcodec/lzo.c

Reimar Doeffinger Reimar.Doeffinger
Wed Jan 31 12:32:17 CET 2007


Hello,
On Wed, Jan 31, 2007 at 11:12:52AM +0100, Guillaume Poirier wrote:
> michael wrote:
> > Author: michael
> > Date: Wed Jan 31 11:07:22 2007
> > New Revision: 7776
> > 
> > Modified:
> >    trunk/libavcodec/lzo.c
> > 
> > Log:
> > 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 ...
> 
> 
> what's more, the code is now more readable for we, poor mortals ;-P

Not overall, the previous way of writing made it more obvious to see the
value ranges that cnt could have in which branch.
It often depends on what you try to understand *g*

Greetings,
Reimar Doeffinger




More information about the ffmpeg-cvslog mailing list