[FFmpeg-devel] [PATCH] Add support for pcm_*_BE in mov

Michael Niedermayer michaelni at gmx.at
Thu Oct 4 21:21:42 CEST 2012


On Tue, Oct 02, 2012 at 11:21:29AM +0100, Tim Nicholson wrote:
> On 01/10/12 20:15, Michael Niedermayer wrote:
> > On Mon, Oct 01, 2012 at 05:25:16PM +0100, Tim Nicholson wrote:
> >> Currently bit depths >16 are only handled correctly if they are LE.
> >> This patch adds support for BE variants and fixes ticket #1767
> >>
> >> Note I have not renamed the function in this patch which suggests a test
> >> for le variants only. Updating this to reflect the general nature of the
> >> function can be done in a separate cosmetic patch if required.
> > 
> > this patch breaks fate
> > [...]
> 
> Updated patch attached.
> This includes new checksums to reflect the change in the mov files due
> to the different header type.
> 
> -- 
> Tim
> 
> 

>  libavformat/movenc.c            |   26 +++++++++++++++++++++++---
>  tests/ref/fate/acodec-pcm-s24be |    4 ++--
>  tests/ref/fate/acodec-pcm-s32be |    4 ++--
>  3 files changed, 27 insertions(+), 7 deletions(-)
> d8b5119fa25913a3a6d760a88e4b559910597e96  0001-movenc.c-Add-support-for-16bit-BE-flavours.patch
> From 4c16fe54575a52334d4aec8d066fa51eaebd0f7b Mon Sep 17 00:00:00 2001
> From: Tim Nicholson <Tim.Nicholson at bbc.co.uk>
> Date: Tue, 2 Oct 2012 11:18:04 +0100
> Subject: [PATCH] movenc.c: Add support for >16bit BE flavours
> 
> Currently only LE handled correctly
> Updated fate checksums to reflect new headers

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121004/73269da7/attachment.asc>


More information about the ffmpeg-devel mailing list