[Ffmpeg-devel] [PATCH] Make avformat_get_context_defaults static

Matti Hamalainen mhamalai
Fri Sep 8 12:59:53 CEST 2006


On Fri, 8 Sep 2006, Michael Niedermayer wrote:

> 
> shouldnt it be
> LIBAVCODEC_VERSION_INT >= ((52<<16)+(0<<8)+0)
> 

Is there some specific reason, why there is no macro
used for this, in order to avoid similar mistakes?

#define MKVERSION(a,b,c) ((a)<<16+(b)<<8+(c))
...
#if LIBAVCODEC_VERSION_INT >= MKVERSION(52,0,0)

or similar?

Of course it's a pretty simple element, but the same
things are repeated through the code, a macro might
simplify potential future changes.

-- 
] ccr/TNSP^DKD^pWp :: ccr(at)tnsp(dot)org :: http://ccr.tnsp.org/
] Fingerprint: 0466 95ED 96DF 3701 C71D   D62D 10A6 28A6 1374 C112




More information about the ffmpeg-devel mailing list