Prepare for UPnP songs that retain there database identity.

Detach the Song class completely from the public API, only to be used
by SimpleDatabase and the update thread.

Reduces overhead because we need to manage only one memory allocation.
According to valgrind/massif, we save 7%.

From now on, struct Song will be used by the database only, and
DetachedSong will be used by everybody else.  DetachedSong is easier
to use, but Song has lower overhead.

Fixes regression from commit e96779de.