[FFmpeg-devel] [PATCH]VDPAU patch for VC1 decoding, round 1

Måns Rullgård mans
Mon Jan 12 00:21:17 CET 2009


Aurelien Jacobs <aurel at gnuage.org> writes:

> Aurelien Jacobs wrote:
>
>> 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.
>
> It seems there is no objection to this patch anymore ?

I already said I objected.  Do I have to repeat it if you post a new
patch doing the same thing?

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list