[FFmpeg-devel] [PATCH]VDPAU patch for VC1 decoding, round 1
Thu Jan 8 01:03:26 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, and more consistent especially
> in cases such as:
> #if ENABLE_A
> #elif ENABLE_B
> #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
iam fine with the patch
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel