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

Diego Biurrun diego
Thu Jan 8 01:09:58 CET 2009


On Thu, Jan 08, 2009 at 12:50:09AM +0100, 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,

Because you save 3 characters?  Hardly...

> and more consistent especially
> in cases such as:
>   #if   ENABLE_A
>   #elif ENABLE_B
>   #endif
> or
>   #if ENABLE_A || ENABLE_B

You missed the #endif in the second part.

But these cases have different semantics, so they are not
comparable at all.

> 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.

How so?

Diego




More information about the ffmpeg-devel mailing list