[FFmpeg-devel] sdp.c and CONFIG_RTP_MUXER

[Andy Parkins]

On Tuesday 2007 July 31, Michael Niedermayer wrote:

> i prefer the #ifdef over if() as later is just not correct its pure luck
> that it works
> also here theres no difference in complexity, the if() is not simpler

Not directly related to this problem, but the common reason for doing things 
like:

  #define FEATURE_ENABLED 1
  if( FEATURE_ENABLED )
    // do something
  else
    // do something else

Is that the syntax is checked regardless of the feature being enabled or not.

Of course - your point about it relying on the compiler to optimise it away is 
correct; but I just wanted to point out that there are advantages to the if() 
method too.



Andy

-- 
Dr Andy Parkins, M Eng (hons), MIET
andyparkins at gmail.com