[FFmpeg-devel] [RFC] remove lrintf fallback implementation
Rich Felker
dalias
Tue Oct 23 03:46:25 CEST 2007
On Tue, Oct 23, 2007 at 12:20:43AM +0200, Diego Biurrun wrote:
> On Mon, Oct 01, 2007 at 03:55:18PM +0200, Diego Biurrun wrote:
> > In libavutil/internal.h we have a (bad) fallback implementation of
> > lrintf that was added waaaay back in October 2002 by Fabrice:
> >
> > static av_always_inline long int lrintf(float x)
> > {
> > return (int)(rint(x));
> > }
> >
> > I'm not sure which systems still lack lrintf, but we refused to add a
> > fallback implementation of llrint for Cygwin. So IMO the lrintf
> > fallback should go.
>
> So nobody has an opinion about this matter (as opposed to an opinion
> about llrint)? If nobody makes a statement in either direction I will
> drop this issue, albeit reluctantly...
As long as the implementation is generic and not asm, I prefer to
leave it. It's not harming anyone and might be useful.
Rich
More information about the ffmpeg-devel
mailing list