[FFmpeg-devel] sdp.c and CONFIG_RTP_MUXER
Luca Abeni
lucabe72
Tue Jul 31 13:05:45 CEST 2007
Hi Michael,
On Tue, 2007-07-31 at 11:49 +0200, Michael Niedermayer wrote:
[...]
> > > isn't it better to use ?
> > > #ifdef ENABLE_RTP_MUXER
> > > {
> > > ...
> > > }
> > > #else
> > > return NULL;
> > > #endif
> >
> > This was my first try... It works, but it generates warnings because
> > some static functions are defined but not used in sdp.c. But I am ok
> > with this version too, if it is preferred.
>
> 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
Ok, I'll prepare a patch using #ifdef (I will need to #ifdef out all the
static functions to avoid warnings), and I'll send it later.
Thanks,
Luca
More information about the ffmpeg-devel
mailing list