Signed-off-by: Rosen Penev <rosenp@gmail.com>

The ChainFilter class is extremely complicated code, and will grow to
be even more complicated when the Filter interface gets extended.
Let's just remove it; we can easily chain many TwoFilters instead.

release v0.22.11

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

Insert an "aformat" filter which converts double-precision to
single-precision.

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

If DetectFilterOutputFormat() fails to determine the output format,
insert an "aformat" filter which attempts to force a specific output
format.

Fixes part 2 of of https://github.com/MusicPlayerDaemon/MPD/issues/1235

Merge some duplicate code.

Since AVFilterContext are freed automatically, this wrapper class
serves no purpose.  Let's remove it.

Fixes potential double-free bugs which currently did not occur because
the destructors happened to be called in the right order.

Fixes part 1 of https://github.com/MusicPlayerDaemon/MPD/issues/1235

The type is duplicated otherwise

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

Found with readability-const-return-type
Signed-off-by: Rosen Penev <rosenp@gmail.com>

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

loop is wrong anyway. It's missing a break;
Signed-off-by: Rosen Penev <rosenp@gmail.com>

The latter is easier to read and is the "correct" thing to do.
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Prevent a segfault when accessing album art.

Fix #1224 #1225

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Found with cppcoreguidelines-avoid-non-const-global-variables
Signed-off-by: Rosen Penev <rosenp@gmail.com>