Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

Signed-off-by: Pierre d'Herbemont <pdherbemont@videolan.org>

module_list_get(): gets the list of modules
module_list_free(): releases the list

Modules have no threads, no variables (!= config items) and
no plugin instances...

When dropping on the main controller => Play
When dropping on the playlist => Enqueue.

If you go under 2x2, it can go seriously bersek.

PFff :D
(cherry picked from commit 65c2a9bf053f82e4a84192f530f042209a449cca)
(cherry picked from commit 834bd87073e19b49dca9d1ea608541e5885fb3cf)
(cherry picked from commit 6a02accb4ca190a3e2b2690f2130ac6a09d78a32)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

1/ pgp v4 signature checks of status files with more than 506 bytes would fail
2/ pgp v4 signature checks would fail (code dupication sucks).

This code is quite a mess. We shouldn't need to implement all this lowlevel stuff. I'll remove most of it and use pgpme instead.

For the time being, this means that upcomming releases should use v3 signatures for the downloaded files and shouldn't use status files of more than 506 bytes (or maybe shouldn't use v4 signatures for status files either)

services_discovery: Partially fix service discovery. We yet need to find a way to properly get the localized name... grr.

Previously we properly initialized the localized name of the object in Open(). Now Open() also runs Run() so we are kind of stuck. Please fix.

As discussed on IRC. Yield was quite confusing.

By the way, the event management seems quite broken. Events may occur
before _Create returns (even before this commit)...

Avoid objects being detached asynchronously too early.
Side effect is that you cannot free your children in the destructor
(that would be a reference loop).