The former was deprecated with C++14.

According to the C++11 and C++17 standards, both files are identical.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

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 modernize-make-unique
Signed-off-by: Rosen Penev <rosenp@gmail.com>

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.

The scan methods must be "noexcept".

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

The libWavPack documentation says:

 "After a FALSE return the file should not be accessed again (other
 than to close it); this is a fatal error."

Technically, these samples are signed.  There was no practical
difference, only the declaration was formally wrong.