[Ffmpeg-cvslog] r7175 - in trunk: configure libavformat/allformats.c

[Måns Rullgård]

Aurelien Jacobs <aurel at gnuage.org> writes:

> On Wed, 29 Nov 2006 11:36:20 +0100
> Baptiste Coudurier <baptiste.coudurier at smartjog.com> wrote:
>
>> Hi
>> 
>> aurel wrote:
>> > Author: aurel
>> > Date: Mon Nov 27 13:55:24 2006
>> > New Revision: 7175
>> > 
>> > Modified:
>> >    trunk/configure
>> >    trunk/libavformat/allformats.c
>> > 
>> > Log:
>> > cleanup the #ifdef mess in muxer/demuxer registration
>> > 
>> > [...]
>> >
>> > -    av_register_output_format(&avi_muxer);
>> > +    REGISTER_MUXDEMUX(AUDIO, audio);
>> >  #endif
>> > +    REGISTER_MUXDEMUX(AVI, avi);
>> >  #ifdef CONFIG_AVISYNTH
>> >      av_register_input_format(&avisynth_demuxer);
>> >  #endif
>> > -#ifdef CONFIG_AVS_DEMUXER
>> > -    av_register_input_format(&avs_demuxer);
>> > -#endif
>> > -#ifdef CONFIG_CRC_MUXER
>> 
>> Any reason for leaving avisynth in old style ?
>
> Yes, because, it wasn't surrounded by #ifdef CONFIG_AVISYNTH_DEMUXER,
> and I wanted to keep the exact same behavior.
> It should probably be changed to depends on ENBALE_AVISYNTH_DEMUXER
> in addition to CONFIG_AVISYNTH.
> But then, should ./configure disable AVISYNTH_DEMUXER if CONFIG_AVISYNTH
> is not set ?
> Is the attached patch ok ?

This should be done in a generic way for all formats and codecs that
depend on external libs.  Leave it as is for now, and I'll look into
it later.

-- 
M?ns Rullg?rd
mru at inprovide.com