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

[Reimar Döffinger]

Hello,
On Wed, Jan 31, 2007 at 10:54:44PM +0100, Michael Niedermayer wrote:
> On Wed, Jan 31, 2007 at 09:57:30PM +0100, reimar wrote:
> > Author: reimar
> > Date: Wed Jan 31 21:57:30 2007
> > New Revision: 7786
> > 
> > Modified:
> >    trunk/libavcodec/lzo.c
> > 
> > Log:
> > Remove boundary checks that are actually done "well enough"
> > in copy function
> [...]
> > @@ -228,11 +224,8 @@
> >          copy_backptr(&c, back, cnt + 2);
> >          state=
> >          cnt = x & 3;
> > -        if (cnt)
> > -            copy(&c, cnt);
> > +        copy(&c, cnt);
> 
> have you benchmarked this? i tried the very same change a few hours ago and
> with nirvana.nuv it was slower

I admit I did not with nirvana.nuv but with some other random picture
and binary files.
That change alone made no difference (was not slower either) but with
the simplification of the checks in copy() it was faster.

Greetings,
Reimar D?ffinger