[MPlayer-cvslog] CVS: main/libaf af_format.c,1.26,1.27

[Oded Shimon]

On Tuesday 22 February 2005 17:36, Michael Niedermayer wrote:
> Hi
>
> On Tuesday 22 February 2005 03:13, Richard Felker CVS wrote:
> > CVS change done by Richard Felker CVS
> >
> > Update of /cvsroot/mplayer/main/libaf
> > In directory mail:/var2/tmp/cvs-serv11864/libaf
> >
> > Modified Files:
> >  af_format.c
> > Log Message:
> > finally the dreaded white-noise-with-floats bug is fixed!!!!
> >
> > the problem is that lrintf was not prototyped on some systems, but
> > it's easier and faster just not to use it at all. looks like the cola
>
> did you benchmark this change, AFAIK lrintf() should be MUCH faster then
> casting to int, as the later needs to change the FPU rounding mode several
> times

Benchmarked, for 5 million samples of ram garbage, lrintf took 0.5 seconds, 
cast took 0.9 seconds. lrintf proved itself to be risky, so for an 
optimization of this size, imho it's not worth it.

 -ods15