[FFmpeg-devel] sdp.c and CONFIG_RTP_MUXER
Michael Niedermayer
michaelni
Tue Jul 31 15:25:14 CEST 2007
Hi
On Tue, Jul 31, 2007 at 01:05:45PM +0200, Luca Abeni wrote:
> 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.
no, i wont accept #ifdef hell to supress warnings!
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070731/739c45dd/attachment.pgp>
More information about the ffmpeg-devel
mailing list