[vlc-devel] Factor HTTP/HTTPS/RTSP port in core

Rémi Denis-Courmont remi at remlab.net
Sun Aug 14 20:31:56 CEST 2011


Le dimanche 14 août 2011 21:09:24 Pierre Ynard, vous avez écrit :
> After these changes, sout blocks get a little funny:
> 
> #http{mux=ts,dst=<stuff that'll be ignored>/<path>}
> #rtp{sdp=rtsp://<more ignored stuff>/path}
> 
> Isn't that a concern? Do you have patches planned to rework that too? As
> a last rework, the NEWS could document these caveats-to-be.

I agree that the syntax is odd. It already was odd before.
I am fine simplying it to just path, but it would fully break syntactical 
backward compatibility.

> Another thing:
> 
> +    add_integer( "rtsp-port", 5554, RTSP_PORT_TEXT, RTSP_PORT_LONGTEXT,
> true ) +        change_integer_range( 1, 65535 )
> 
> I couldn't guess whether you meant 554, or used 5554 instead of the
> registered rtsp-alt port 8554.

I don't really care. 554 would cause a lot of problems as stupid users don't 
understand the concept of "Permission denied" error on privileged ports.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis



More information about the vlc-devel mailing list