[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