removed unneed check for protocol in shout plugin, will assume icecast2 protocol if not exist in config

g_error_free() was missing in case g_convert() failed.

directory_free() should free all of its children (subdirectories and
songs).  This way, db_finish() properly frees all allocated memory.

"LOG_H" is a macro which is also used by ffmpeg/log.h.  This is
ffmpeg's fault, because short macros should be reserved for
applications, but since it's always a good idea to choose prefixed
macro names, even for applications, we are going to do that in MPD.

After a partial write, chances are vanishing that another write() will
succeed.  Don't try immediately.

The "simple singly-linked-list" library has been replaced with GLib's
GList and GQueue.

Another custom data structore converted to GLib.

Replace a custom data structure with a GLib one.

The list cache aims to save memory allocations, and complicates the
code a bit.  We should rather use GLib slices later, which are easy to
use.

The source _ogg_common.c does not need any symbols from
_flac_common.h, but including it leads to compiler errors when libflac
isn't available.

Again, I forgot to adapt oggflac to the new API (struct input_stream,
bool return values).

decoder_data() uses wait times to let the input stream continue its
transfer.

decoder_data() always returns the current command.  If we use this, we
can save a lot of decoder_get_command() calls.

Converted the runtime check to an assertion.

Remove one indent level.

Move code from ffmpeg_decode_internal() to make it smaller and more
readable.

The decoder API provides the function decoder_seek_error() to report
seek errors.  Use this function instead of logging the error.

The function mpdurl_read() is too complicated, and uses the wrong data
types.

tag objects must be freed with tag_free() to ensure that all resources
are freed.

The function ffmpeg_tag() already has the variable base.tag, which can
be used for this.

ffmpeg_try_decode() did not interpret ffmpeg_helper()'s return value
properly; migrate everything to bool to make it consistent.

We don't need those anymore, they just fill the log.

ffmpeg_tag() did not initialize base.decoder, which made valgrind
unhappy, and can lead to a egmentation fault.

Depending on MPD's umask, the file permissions of the unix socket were
too restrictive, and many clients were not able to connect.  Do a
chmod(0666) on the socket, to allow everybody to connect.

Commit d692e9 broke configurations without explicit server or sink
configuration.  Check for getBlockParam()==NULL.

calling aclocal -I PWD/m4 in autogen.sh, rather than aclocal -I m4 is
causing autotools to to not include the m4 directory in the dist
tarball. This makes it quite hard to regenerate aclocal/configure.

Similar to libmad, libmpcdec provides samples with higher quality than
16 bit.  Send 24 bit samples to MPD, which allows MPD to apply
dithering just in case the output devices are only 16 bit capable.

Don't hard-code the "16 bits" or "2 bytes" in multiple locations.

The name "s16" implies 16 bit integers.  To make the code more
generic, rename it to "dest".

The conversion of integer samples was completely broken, which
presumably didn't annoy anybody because libmpcdec provides float
samples on most installations.

When input_stream_seek() was converted to return a bool, this wasn't
adjusted in the musepack plugin.

Don't return 0/-1 on success/error, but true/false.  Instead of int,
use bool for storing flags.

Its only caller in mp3_decode() just compared its value with
DECODE_BREAK.  Convert that to bool, and return false if the loop
should be ended.  Also eliminate some superfluous command checking
code, which was already done in the preceding while loop.

Since open() and play() close the device on error, we can simply check
audio_output.open instead of audio_output.result after a call.

When one of several output devices failed, MPD tried to reopen it
quite often, wasting a lot of resources.  This patch adds a delay:
wait 10 seconds before retrying.  This might be changed to exponential
delays later, but for now, it makes the problem go away.

Moved code from syncAudioDeviceStates() to audio_output_update().

Stopping an audio output device without cancelling its buffer doesn't
make sense.  Combine the two operations, which saves several cancel
calls.

Don't allow users to open a file which is non-regular (e.g. pipes,
devices).