[FFmpeg-devel] [PATCH] Add SAMPLE_FMT_S24 audioconverter

Ian Caulfield ian.caulfield
Mon Aug 4 09:29:10 CEST 2008

2008/8/4 madshi <dear at madshi.net>
> Michael Niedermayer schrieb:
>  > Iam against this patch.
>  > The code that stores 24bit PCM in int32_t should shift it
>  > left appropriately so it is SAMPLE_FMT_S32.
> But in that case the information that the audio data is only
> 24bit and not 32bit is lost, isn't it? Sometimes it's useful
> to know that only 24bit of the audio information are filled
> with useful data.

There is a field in the codec context indicating the number of bits
per sample - I think the MLP decoder is the only audio decoder that
uses it though. The advantage of this field is that it can indicate
e.g. 18- or 20-bit audio, not just 8,16, 24, 32.


More information about the ffmpeg-devel mailing list