[FFmpeg-devel] [PATCH 4/4] Include headers for optional libraries only when enabled.

Diego Biurrun diego
Sun Oct 12 17:52:51 CEST 2008


On Thu, Oct 02, 2008 at 09:26:03PM +0200, Aurelien Jacobs wrote:
> Diego Biurrun wrote:
> 
> > On Thu, Oct 02, 2008 at 06:55:19PM +0200, Diego 'Flameeyes' Petten? wrote:
> > > Include the headers for libavfilter, libswscale and libpostproc when
> > > they are enabled (and thus linked in).
> > > 
> > > --- a/cmdutils.c
> > > +++ b/cmdutils.c
> > > @@ -26,10 +26,16 @@
> > >  
> > > +#if ENABLE_AVFILTER
> > 
> > You should use CONFIG_ definitions in preprocessor checks.
> 
> Why ?

Consistency.  We do have two different definitions, it was my impression
that they should be used for different purposes.

Diego




More information about the ffmpeg-devel mailing list