Simplify the definition of many build options.

Use the new uri_get_suffix() overload that removes the query string.

For free().

Convert filesystem charset to UTF-8 for playlist_list_open_uri().
This fixes one of many remaining charset bugs.

The name "cue" was listed twice in "mpd --version".

Finally allows loading playlist files on a storage plugin.  Commit
297e2747 attempted to implement this, but failed due to this bug.

Necessary for "final" on gcc 4.6.

Let the caller decide whether to strip.  Only remove \n and \r
(end-of-line markers).

Revert to the old API before commit e9e55b08, removing unnecessary
bloat.

Now that Song::GetDuration() is used (which considers start_ms and
end_ms), we don't need to override the Tag's duration value.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Replaces the method Close().

Eliminates some overhead and some duplicate code, and fixes a serious
bug: the old code did not append the ".m3u" suffix, and thus the
"load" command was completely broken for stored playlists.  D'oh!

Playback with some decoder plugins was broken because the Queue's
DetachedSong instances did not have the "real" URI (the mapped path).

Simplifies a lot of code, because we don't need to return both the
SongEnumerator and the InputStream.

There was quite a lot of duplicate code for loading DetachedSong
objects, with different semantics for "securely" loading local files.

Merge duplicate code.