Introduced in C++17. It replaces gcc's warn_unused_result.

Found with modernize-use-nodiscard.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Found with readability-redundant-control-flow
Signed-off-by: Rosen Penev <rosenp@gmail.com>

The formula did not consider the channel count.

Closes https://github.com/MusicPlayerDaemon/MPD/issues/639

Add a `constexpr` constructor and several `constexpr` methods to
construct a DecoderPlugin at compile time, in a way which allows
adding new methods later without having to edit each plugin.

Eliminates a number of allocations, because callers don't need to copy
the strings to a newly allocated buffer only to null-terminate them.
And most callers don't need to have a null-terminated string.

Variable-length arrays are a C-only feature.

These are used by DSD-streaming servers.  For example, MiniDLNA uses
"audio/x-dsd".

The scan methods must be "noexcept".

Replaces decoder_data() and others.

Replaces decoder_initialized().

Prepare for a Decoder API redesign based on an abstract class with
virtual methods.

The old struct config_param remains only for top-level string options.

Check for STOP before decoding the first chunk.  This reduces the
command latency.

Remember the chunk's total size.