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

Måns Rullgård mans
Mon Jan 12 00:23:01 CET 2009


Diego Biurrun <diego at biurrun.de> writes:

> On Mon, Jan 12, 2009 at 12:12:41AM +0100, Aurelien Jacobs wrote:
>> 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 ?
>> If so, I will commit soon.
>
> The patch replacing all CONFIG_s with ENABLE_s?
>
> I don't like it.  However, we can discuss a patch that gets rid of
> the ENABLE_s.

Are there any uses of #ifdef that cannot be converted to #if?  I'm not
sure what that would be, but I'm not sure they can't exist.

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




More information about the ffmpeg-devel mailing list