A destructor would never have to release its own object, since the
destructor is only called when the object is not referenced anymore.
On the other hand, it is not clear to me whether SD objects still need
a destructor.

Otherwise, if the interface is hidden behind another window, clicking
the tray icon however many times will request user attention in the task
bar and never actually show the interface. This was confusing and
inconvenient (fixes #2396). This new behavior is consistent with KMail.

It would probably be even better to show the interface on top (instead
of hiding it, then show it on the second click). Unfortunately, while it
does seem possible, I do not know how. For sure, the previous scheme did
not work anyway.

Initial patch from Doug West.

It fixes support for VC-1 and Theora and Vorbis in avcodec decoder.

This reverts commit 1eefd3dd.
This fixes GLX crashing while requesting an embedded video window.

Conflicts:

	src/video_output/vout_intf.c

Signed-off-by: Rémi Denis-Courmont <rdenis@simphalempin.com>

On Win32, vsnprintf() and vsprintf() are formatting certain things
differently, e.g. "%p". We need to use the same implementation
throughout our vasprintf() replacement.
Signed-off-by: Rémi Denis-Courmont <rdenis@simphalempin.com>

It fixes embed video at start.

Pointed out by Courmish.

It eases security review.

It fixes item name with xpsf playlist (at least).

Otherwise the receiver would -in principle- try to receive RTP on the
next (plus one) even port, whereas the m= line port would be used for
RTCP. We could also pass a=rtcp for the normal even/RTP & odd/RTCP
ports setup, but that can only confuse receivers with no gain.

Note that receivers may not parse the a=rtcp parameter in any case. In
fact, VLC does not parse the a=rtcp field to this day. In any case,
sending RTP to an *even* port is the only safely interoperable option.