[FFmpeg-devel] [PATCH]VDPAU patch for VC1 decoding, round 1
Aurelien Jacobs
aurel
Thu Jan 8 00:50:09 CET 2009
Diego Biurrun wrote:
> On Wed, Jan 07, 2009 at 10:50:39PM +0100, Carl Eugen Hoyos wrote:
> >
> > --- libavcodec/vc1.c (revision 16481)
> > +++ libavcodec/vc1.c (working copy)
> > @@ -4317,3 +4338,35 @@
> > +
> > +#if ENABLE_WMV3_VDPAU_DECODER
>
> Using ENABLE_ here is still wrong...
I disagree. Using #if ENABLE_ is better than #ifdef CONFIG_.
It is more readable, shorter, and more consistent especially
in cases such as:
#if ENABLE_A
#elif ENABLE_B
#endif
or
#if ENABLE_A || ENABLE_B
Moreover, #if gives a chance to the compiler to complain if
there is a typo in the symbol name. With #ifdef, such a
typo could stay unnoticed for a long time.
Now I expect some answer about consistency because the code
supposedly contains more occurence of #ifdef CONFIG_ than
#if ENABLE_.
The attached patch solves this by replacing all usage of
CONFIG_ by ENABLE_.
With this applied, we could also drop all CONFIG_ from
config.h.
Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config_enable.diff
Type: text/x-patch
Size: 118141 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090108/659c79ae/attachment.bin>
More information about the ffmpeg-devel
mailing list