...rather than when disabling it. Otherwise service discovery plugins
cannot possibly cleanup after themselves (and CABOOM).

Whatever would need this would be a buggy SD plugin.

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.