Track whether _finish() has been called, and deliver all partial
results then.  Fixes Mantis ticket 0003621.

Rationale: vanilla libid3tag does not have any pkg-config stuff
and fails to detect because symbols from libz are not found.

- fix potential memory leak of system_path

  'Potential' because currently g_get_system_config_dirs()
  returns single entry on Windows, but that might change.

- remove incorrect g_free() call

  It's not required at all because
  g_get_system_config_dirs() returns GLib owned memory.

- remove extra semicolon

Since some configurations use the "bind_to_address" option in their
/etc/mpd.conf, the systemd service file must wait for the
network.target or else mpd will start before it and thus fail due to
no iface.

The song's URI must be UTF-8, not filesystem character set.

Fixes minor Windows compatibility problem.

Implements Mantis feature request 3582.

Interleaving was completely wrong.  This code was never used, so it
didn't have an effect.

Works around assertion failure in decoder_timestamp().

Use av_get_sample_fmt_string() to obtain a human-readable string.

API simplification.

This is necessary for Ogg packets that span more than one page.

We can only check for negative values if the variable is signed.

Make sure the file header gets written at the beginning, before
_write() gets called.

Move the raw buffer to struct shout_data.

Make it a local variable instead.

These are confusing, and since MixRamp development has ceased, not
useful to anybody.

libvorbisidec and libvorbis export the same symbols, which is a
dangerous thing.  Since libvorbisenc depends on libvorbis, this can
get nasty, so let's disable the Vorbis encoder unless the user
explicitly wants it.

WavPack tags are always APEv2, by definition.  Reuse the tag_table
from tag_ape.c, instead of rolling our own.

Avoids clang pickiness.  This code is not correct, but we'll fix that
another day.

Fixes potential access to uninitialised memory.