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

Aurelien Jacobs aurel
Sun Oct 12 18:48:47 CEST 2008


On Sun, 12 Oct 2008 17:52:51 +0200
Diego Biurrun <diego at biurrun.de> wrote:

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

This file already contains some #if ENABLE_*, and it is shorter
than #ifdef CONFIG_*.

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

Maybe we should try to reduce usage of the useless one
(ie. CONFIG_*), with a final goal being totally removing one
of the two definitions ?
That would slightly simplify the code, overall.

Aurel




More information about the ffmpeg-devel mailing list