[FFmpeg-devel] FFMPEG RTSP parameters support

Martin Storsjö martin
Mon Nov 15 16:09:08 CET 2010


On Mon, 15 Nov 2010, Ronald S. Bultje wrote:

> On Mon, Nov 15, 2010 at 2:19 PM, Martin Storsj? <martin at martin.st> wrote:
> > On Mon, 15 Nov 2010, Stas Oskin wrote:
> >> > Stas, can you test the patch with the actual server where this was needed?
> >> > You could also remove the else branch from the addition in rtspdec.c and
> >> > see if that is enough for it to work.
> >> >
> >> >
> >>
> >> Patch works great, both originally and with the else section removed:
> >> + ? ?} else {
> >> + ? ? ? ?/* Remove the query parameters from the control uri for subsequent
> >> + ? ? ? ? * requests. */
> >> + ? ? ? ?char *ptr = strchr(rt->control_uri, '?');
> >> + ? ? ? ?if (ptr)
> >> + ? ? ? ? ? ?*ptr = '\0';
> >> + ? ?}
> >>
> >> Thanks for the fix, hope it get committed to tree.
> >
> > Great, thanks for testing.
> >
> > Ronald, Luca B, ok to commit without that else branch?
> 
> Yes, thanks :-).

Ok, applied.

// Martin



More information about the ffmpeg-devel mailing list