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

[Luca Abeni]

Hi Ronald,

Ronald S. Bultje wrote:
[...]
>> Some data points:
>> 1) Before r10605, rtsp://localhost:5454/test1-rtsp.mpg?tcp resulted in
>> path="/test1-rtsp.mpg?tcp"
>> 2) After r10605, the same url produced path="/test1-rtsp.mpg?tcp", so no
>> change for this test case...
>> 3) r10775 "fixed" something so that the result is now
>> path="/test1-rtsp.mpg?".
>> 4) So, I tried a different url: rtsp://localhost?tcp. The results are
>> path="?tcp" before r10605, path="" after r10605, and path="" after
>> r10775. Again, that commit does not seem to fix any regression.
> 
> 
> That one (w/o the / in the URL) is supposed to be fixed, I remember testing
> that (we were actively working on ipv6 issues), and receiving confirmation
> [1] that others saw it fixed too. Did I mess up that badly?

I investigated the problem a little bit more, and it seems that: the 
"?..." options must be removed from the host name, but not from the path 
(also see last message from Michael). r10775 fixed host parsing (which 
was causing the ffplay regression), but broke path parsing (by removing 
"?..." from it), and noone noticed that.

Now, Michael's patch seems to fix almost everything (I only found a 
suspicious case that I am testing).


			Thanks,
				Luca