[FFmpeg-devel] [PATCH][RFC] Lagarith Decoder.

Michael Niedermayer michaelni
Wed Aug 12 19:29:05 CEST 2009


On Wed, Aug 12, 2009 at 06:31:36PM +0200, Reimar D?ffinger wrote:
> On Wed, Aug 12, 2009 at 05:10:36PM +0100, M?ns Rullg?rd wrote:
> > Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> > 
> > > On Wed, Aug 12, 2009 at 04:12:25PM +0100, M?ns Rullg?rd wrote:
> > >> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> > >> 
> > >> > Well, you still have to teach the compiler at least to use clz for
> > >> > av_log2, I think you haven't yet ;-)
> > >> 
> > >> I can't because it's in common.h, which is installed.  We really
> > >> should find a way to fix that.
> > >
> > > Just put the optimizations under HAVE_AV_CONFIG_H like everything else
> > > in there that is messy?
> > 
> > And keep adding to the mess?  That's not the way to do things.
> 
> Then how about adding a new internal header with that function and
> putting the version in the installed header under
> #ifndef HAVE_AV_CONFIG_H
> i.e. just to keep API compatibility?
> Possibly even deprecating that version?
> Would that be clean enough?

It would be nice if applications using libav* could also use the optimized
version

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090812/fa2345b5/attachment.pgp>



More information about the ffmpeg-devel mailing list