[vlc-devel] commit: Qt4: remove proto combo from open net tab ( Rémi Denis-Courmont )

[Rémi Denis-Courmont]

Le mercredi 25 novembre 2009 22:48:38 Jean-Baptiste Kempf, vous avez écrit :
> On Wed, Nov 25, 2009 at 09:38:09PM +0100, git version control wrote :
> > vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Wed Nov 25
> > 22:35:23 2009 +0200| [3ae0e266205878bd66b004ee23b40b2c2433b74e] |
> > committer: Rémi Denis-Courmont
> >
> > Qt4: remove proto combo from open net tab
> >
> > Now, we get the URL directly. This is much simpler and much less
> > confusing. Also a few missing network protocols.
> 
> Well, I disagree.

You said the exact opposite 9 months ago, i.e. we should keep the two separate 
to support Stream/Record. I am actually all for removing the network entry 
from the Media menu.

> We already have a Ctrl+V box, if we want.
> If people want to write an URL and then select the port, they are
> screwed

There is a port field in URLs, for non-default ports.

> up now

They were screwed since the port box was confusingly greyed out so far.

Indeed, it only worked with RTP and UDP, nevermind that those are seldom used, 
and even more seldom used manually. Typically, you'd get RTP address via SAP, 
SDP files, or playlists.

> and need to add the proto and not mistake the "://"
> stuffs...

Nobody does know that as it's a VLC-specific syntax. That is why the dialog 
was unusable. Also the "Address" caption was totally wrong since VLC actually 
expected everything but the URL scheme (host name, port number, file path...).


-- 
Rémi Denis-Courmont
http://www.remlab.net/