[FFmpeg-devel] [PATCH] Move ff_reverse to libavutil

Måns Rullgård mans
Mon Nov 9 15:39:54 CET 2009


"Ronald S. Bultje" <rsbultje at gmail.com> writes:

> Hi,
>
> 2009/11/9 M?ns Rullg?rd <mans at mansr.com>:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>>> we can optimize it if we put the part of config.h that specifies
>>> the architecture in a installed header
>>
>> This is not an option. ?We've had this discussion before. ?Summary:
>> config.h depends on the compiler used to build FFmpeg. ?Users of the
>> installed headers might be using a different one.
>
> You can split it and put the compiler-independent stuff
> (CONFIG_*_DE/ENCODER etc., and arch) in an installed header and the
> rest in a non-installed one, or make all that depend on available
> headers / header variables such as __GNUC__ or so.

The optimisations we're talking about depend on compiler-dependent
bits, e.g. HAVE_INLINE_ASM.  Testing for __GNUC__ isn't enough.

> I'm sure this would be useful for something like VLC as well, to know
> which decoders are available when a user compiled it himself.

The encoders can be enumerated through the API.  I guess a static
check might be useful too, of course.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list