A <track> can have 0 or more <location> elements. VLC doesn't like elements without a uri however, so init an element with vlc://nop. Note also that we currently only support 1 location element per track....

This closes #2891

This ensure consistency with simple Open.
Moreover, with those KDE effects, this is cool.

Close #2851
(cherry picked from commit 084a6247a99e90c1ab66090811f2a96f8c816ed7)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

(cherry picked from commit 31d64379a5706513ee6ed4e440c9e723a9842c9e)
Signed-off-by: Jean-Paul Saman <jean-paul.saman@m2x.nl>

(cherry picked from commit f23f5e62dcb29ca9b12a1cac73a00c464dbe6f3d)
Signed-off-by: Jean-Paul Saman <jean-paul.saman@m2x.nl>

only non-deprecated functions.

(Rtp)Add then MuxAdd, so MuxDel before (Rtp)Del. Indeed, the PS muxer
sends packets when it's deleted, so the RTP stream is still needed.

When it is called with strings that contain " " or '\' or don't have ://, we have an indication that the caller is likely providing a path. This will help us find these cases in the future.

Select fullscreen display the one that has most of the video-window,
then you get qt4 fullscreencontroller on same display also, and it's
more logical (imo). Algorithm is prettymuch the same as Nigel Tao has
made in his gpl2 project superswitcher, Didn't want to reinvent this
with different variable-names ;)

skins was using two lists of variables (named and anonymous).
The problem was that variables from one list held references to variables from the other list and vice-versa. Whatever the order of releasing them, crashes could occur.

This patch uses the anonymous list to keep a reference on **all** variables.
This guarantees they are released in the reverse order from creation.

When we fill the name of an input item with URI parts, use decode_URI() so it becomes easier to read.

, like the name indicates.
Pass unmodified URIs from xspf to vlc, and stores them in xspf
unmodified as well

This commit partly reverts [9a746cfa]
Code using psz_uri should be checked for consistency, and GUIs might want to decode components at display

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

SubX crashed on both Linux and Windows in a set::erase call (STL) when releasing C++ objects. Swapping two loops solved it. (the second loop was referencing variables released in the first loop). This makes SubX terminate nicely, but doesn't solve all crashes for skins2. (more cleanup to do)