This reverts commit 26ad82e3.

by a mutex).

The string given in a callback must be copied.

This saves memory, diminishes latency and fixes an unlikely loss of
events if too many actions are sent. This may expose some re-entrancy
bugs, although I did some succesful smoke tests. Any path that emits a
key action or a key press needs to be careful. vout_thread_t.pf_manage
is the most obvious case. That should be safe as the vout thread holds
no locks when in pf_manage. Not sure about OpenGL though.

A small bug with SPU across vout change remains present as before.

When we connect through a proxy, the certificate common name does not
match the TCP-level hostname...

Set the nop at the end of parsing, instead of the beginning, because SetURI will only init the name once. Now we had all media library items with the title "nop".

early).

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 084a6247)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

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

(cherry picked from commit f23f5e62)
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.