[Ffmpeg-devel] [PATCH] add ENABLE_ defines along with CONFIG_

Aurelien Jacobs aurel
Sat Nov 4 18:29:46 CET 2006


Hi,

The attached patch generate ENABLE_ defines for every codec/format/parser
along with the CONFIG_ defines in config.h.
Those new ENABLE_ defines are always defined, either to 1 when the feature
is enabled or to 0 when it's disabled.
It allows to write this:

  if (ENABLE_DVVIDEO_DECODER) {
    [...]
  }

instead of this:

#ifdef CONFIG_DVVIDEO_DECODER
    [...]
#endif

Here are the advantages as I already explained in my previous mail
about "ifdef dv in avi container":
 - smaller source code
 - less ugly #ifdef mess
 - the compiler can still parse and check for errors in the
   DV only sections before optimizing them out

Ok to apply ?

Aurel
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: config-enable.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061104/8f4ec8d6/attachment.txt>



More information about the ffmpeg-devel mailing list