GitLab

Currently libvlc uses "state" callback to notice input state changes, and the playlist notice input state change via direct call from the input.
playlist can't use the "state" variable as it is also used by the playlist to ask the input to do a certain action.

The event approach is much more clear here.

It needs to be done by the parent (VLM or playlist)

This could trigger some interface bugs, especially if they handle their callbacks carelessly. Revert if you have issues, but also consider to fix your interface.

The kept sout needs to be attached to the playlist, else we will
try to destroy vlm's souts before it's time to.

Plugins are already using pl_Yield to get the playlist, as they should.

...and use it for stats and the config lock

Signed-off-by: Christophe Mutricy <xtophe@videolan.org>

Remove VLC_META_ENGINE_*
Don't use VLC_META_INFO_CAT
Avoid using input_Item{Add,Get}Info
Remove disabled code in cdda
Remove "meta fetcher":
    There was only 1 module (not) implementing it (musicbrainz)
    Reduces complexity in playlist core
Use a more consistent naming
Factorize QT4 sorting and column display code
Remove unused members

Caution: other pieces of code may have been randomly deleted in
920692ff

input: Split Init() into multiple subfunctions to try to isolate the different part of the function.

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

Signed-off-by: Rafaël Carré <funman@videolan.org>

stats: Move the stat to libvlc instead of the playlist. As stated in the code it is not playlist related.

input: Make sure stat info are created before attaching to the parent, because stats_ComputeGlobalStats() may vlc_list_find() the input before the stat have been initialized.