Passing it by value is actually smaller (32 bit) than the rvalue
reference (64 bit pointer), and it ensures that the object is consumed
after the call returns, no matter how the methods are implemented.

Migrate callers to GetSocket().Read(), GetSocket.Write(), which is the
same.

Coverity discovered that the Pulse plugin could throw exceptions from
Pause(), but that method was marked "noexcept" because its caller was
not designed to catch exceptions.  So instead of avoiding exceptions
(by catching and logging them in each and every implementation), let's
allow them, and do the catch/log game in the MPD core.

Yet another C-style vtable replaced with C++.

Hide struct FilteredAudioOutput from the plugins, preparing for hiding
MPD's core internals.

Prepare to add an abstract class AudioOutput, to be implemented by
plugins, to get rid of the C-style vtable.

Don't access AudioOutput::pause.

We can remove those C++11 and C++14 kludges because we require GCC 4.9
now.

This eliminates some overhead, because the compiler doesn't need to
consider these functions throwing.

The AudioOutput struct (which is exposed to all plugins) should not be
aware that it's being controlled by another thread.

Fixes a potential deadlock introduced by commit 94528735

Using variable-size objects is not worth the trouble here.  Let's drop
this and use existing and simpler code.

Fixes another buffer overflow: if the stream has a very long title or
URL, resulting in a metadata string of more than 2 kB, icy_string[0]
is a negative value, which gets casted to size_t - ouch!

 https://bugs.musicpd.org/view.php?id=4652

Fixes a buffer overflow due to the bad formula rounding the buffer
size up.  At the same time, remove the "+1" from the meta_length
calculation, which takes the padding into account and at the same time
implements proper rounding.

Why did this cast exist??

Eliminate dependency on io_thread_get().