Its functionality was removed several years ago, so it doesn't do anything.
AVCodecContext.frame_number could serve the same purpose if someone
wants to debug the frame count.

On frame decoding failure, return an error if the frame is the same size as
the whole packet, otherwise just log an error message and return the number
of bytes consumed.

Specify avoptions and how to use pactl to list devices.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

It currently use the simple api and is using the latency information
provided only to offset the stream start.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

this was forgotten when the encoder was removed

avcodec_decode_audio3() already checks it before sending the packet to the
decoder.

Use AVCodecContext.frame_number instead.

This is already checked by avcodec_decode_audio3()

prior to decoding.

blocks.

The values will always be the same, so this change eliminates an unneeded
variable. It also gets rid of the need to reset src when memcpy() is used.

The Zork PCM decoder does not decode the 1 sample we have correctly, therefore
the encoder based on the decoder is also incorrect. There is no good reason to
keep the encoder.

It is already checked by avcodec_open2().

the sample size.

This is already done in avcodec_decode_audio3()

Also return AVERROR_PATCHWELCOME.

They only operate on stereo content, so the extra param is not necessary and
also allows for simplifying the code.

Now they only do stereo interleaving.

This should also fix decoding of mono 24-bit.

It is identical for 16-bit and 24-bit, so there is no need to have duplicate
code.

The bits are not wasted, they are additional low bits that are added to the
16-bit decompressed samples to increase the output sample depth.

Also rearranges some functions for easier cleanup on failure.

reduces memory usage when the stream has fewer than MAX_CHANNELS

check frame header channel count against header/container channel count.

It is already done when using it to set sample_fmt.