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

This partially fixes multiple media players in LibVLC. This also
removes a few run-away var_Create() in doVolumeChanges().

This fixes, e.g. garbage OSD display when toggling mute with the hotkey.

PARENTAL ADVISORY: explicit hack

This should allow volume control to (somewhat) work in LibVLC.
Unfortunately, it will still be buggy when there are multiple players.

This will probably never work for LibVLC.

pl_Hold() assumed the playlist was already created.
pl_Release() does not solve any problem. Playlist synchronization is
done with the playlist lock, while playlist deletion must anyway be done
after all interfaces, services and other threads are joined.

This fixes the following bug:
When scrolling the mouse wheel down, volume jumps from 0% to 400%.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

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

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

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

It will allow to sanetize aout_instance_t.

Destroying directly saves one Lookup.

(and fix a memleak)

 The input event vout now warns about every vout creation/destruction
due to a decoder.

The audio output locking rule seems very vague. Moreover there could be some cross lock issues.

(Should fix a number of crash^W assert report that we had in apple-bugreport ml).

Fixed missing lock in aout callbacks !
Factorized duplicated code.

This improves readability a lot and prevent us from reinventing the wheel.

Missing some cases that I could not test.
Also ffmpeg/chroma.c is locked, so I can but leave it broken.
Fix is for the remaining modules is obvious and the issue is easy to detect.

set volume-change variable also in volumeUp/volumeDown, because it seemed that when you change volume with that, it wasn't setted

Add warning if people try using an unimplemented aout function (zorglub!!!). This is currently only possible through the qt interface (i haven't tried it myself).