[FFmpeg-devel] [PATCH] set bits_per_raw_sample in pcm decoder

Michael Niedermayer michaelni
Sun Jun 6 23:08:20 CEST 2010


On Sun, Jun 06, 2010 at 02:00:13PM -0400, Justin Ruggles wrote:
> Hi,
> 
> The pcm decoder needs to set bits_per_raw_sample when sample_fmt is
> SAMPLE_FMT_S32.  This information can then be passed to the encoder,
> which can make use of it (ffmpeg.c does this).
> 
> In particular, lossless codecs that can encode 24-bit audio need to
> write stream and/or frame values with the bits per sample of the source.
>  Also, the checksums (e.g. MD5 for FLAC, CRC32 for ALS) are based on
> 3-byte source data for 24-bit audio.  The encoder cannot distinguish
> between 24-bit and 32-bit source without having bits_per_raw_sample set.
> 
> -Justin
> 

>  pcm.c |    4 ++++
>  1 file changed, 4 insertions(+)
> dc4586102e2dc3098343c94b2a904419e95ebec5  pcm_bits_per_raw_sample.patch

looks ok if it passes tests

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100606/a09f48b1/attachment.pgp>



More information about the ffmpeg-devel mailing list