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

[D Richard Felker III]

On Tue, Feb 22, 2005 at 08:18:13PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Tuesday 22 February 2005 16: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
> >
> > btw, dont forget -D_ISOC99_SOURCE or -D_ISOC9X_SOURCE otherwise gcc will
> > not inline lrintf()
> 
> or -D_GNU_SOURCE
> 
> so to summarize one of the following is needed -D_ISOC99_SOURCE or 
> -D_ISOC9X_SOURCE or -D_GNU_SOURCE
> 
> many of our Makefiles add -D_GNU_SOURCE, the one from libaf does not

ok, please fix if you think you can...

rich