[FFmpeg-devel] [PATCH] Add an RTSP muxer

Martin Storsjö martin
Thu Jan 7 09:42:42 CET 2010


On Thu, 7 Jan 2010, Luca Abeni wrote:

> Martin Storsj? wrote:
> > On Thu, 7 Jan 2010, Luca Abeni wrote:
> > 
> > > A quick question about patch 04/24: why
> > > -        if (port > 0) {
> > > +        if (dst[0]) {
> > >              s.dst_addr = dst;
> > > -            s.ttl = ttl;
> > >          }
> > > +        s.ttl = ttl;
> > > and not simply
> > > -        if (port > 0) {
> > > +        if (dst[0]) {
> > >              s.dst_addr = dst;
> > >              s.ttl = ttl;
> > >          }
> > 
> > No particular reason, actually, I wasn't really sure of which way to do it,
> > so either way is fine for me.
> 
> Committed the simpler version.

Great, thanks! It's always good to be able to drop some of the patches of 
a longer series.

// Martin



More information about the ffmpeg-devel mailing list