[FFmpeg-devel] [PATCH] rtsp - alternate protocol
Michael Niedermayer
michaelni
Fri Dec 28 21:52:23 CET 2007
On Fri, Dec 28, 2007 at 09:16:49PM +0100, Luca Abeni wrote:
> Hi Michael,
>
> Michael Niedermayer wrote:
> [...]
> >> If the patch is ok, I can commit it in three times (first fix option
> >> parsing, then remove rtsp_default_protocols usage from ffplay, and then
> >> remove such variable from rtsp.c).
> >
> >
> > patch ok if the following hunk is intended
> >
> > [...]
> >
> >> @@ -864,7 +859,7 @@
> >> port = RTSP_DEFAULT_PORT;
> >>
> >> /* search for options */
> >> - option_list = strchr(path, '?');
> >> + option_list = strchr(s->filename, '?');
> Yes, this is needed for unbreaking option parsing (so, without this
> change it is not possible to select a specific transport protocol).
> As far as I can see, RTSP options are currently not detected, because
> url_split() is removing the "?..." part of the URL from path, so
> option_list is always NULL...
> Maybe there has been a change in url_split() and the code in rtsp.c has
> not been updated?
?foobar should be part of the path i think ... url_split() might be buggy ...
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071228/0fe58d7a/attachment.pgp>
More information about the ffmpeg-devel
mailing list