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

Aurelien Jacobs aurel
Wed Nov 29 22:39:15 CET 2006


On Wed, 29 Nov 2006 19:14:56 +0000
M?ns Rullg?rd <mru at inprovide.com> wrote:

> 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.

Indeed that would be better.

> Leave it as is for now, and I'll look into it later.

Fine.

Aurel




More information about the ffmpeg-cvslog mailing list