[FFmpeg-devel] [PATCH] rtsp - alternate protocol

[Luca Abeni]

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?


			Thanks,
				Luca