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

D Richard Felker III dalias at aerifal.cx
Tue Feb 22 21:19:49 CET 2005


On Tue, Feb 22, 2005 at 09:58:49PM +0200, Oded Shimon wrote:
> On Tuesday 22 February 2005 21:53, D Richard Felker III wrote:
> > On Tue, Feb 22, 2005 at 05:29:59PM +0100, Reimar Döffinger wrote:
> > > How about just improving the configure check
> > > (-Werror-implicit-function-declaration maybe?).
> >
> > Perhaps, but imo it's bad to use lrintf anyway -- it's probably slow
> > unless the libc makes it inline.
> Actually, rather amazing part - I just benchmarked lrintf without 
> '-D_GNU_SOURCE' and with -lm, (it gave an implicit declaration warning btw, 
> and I did include math.h) - it was 0.6s! still faster than cast, which is 
> 0.9+s, and only slightly slower than 0.55s inline lrintf.
> 
> I can't really make any decisions about which method is best, but I can tell 
> that those are the benchmarks.

I can also tell you, from our discussion on irc, that your benchmarks
are totally unscientific and bogus. :)

Rich




More information about the MPlayer-cvslog mailing list