[FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

[Michael Niedermayer]

On Fri, Dec 11, 2015 at 11:32:13AM +0000, Mats Peterson wrote:
> OK, here is an improved version of my previous patch of matroskadec.c that manages the palette itself without calling code in mov.c. I have borrowed some code from it, though, and I'm including qtpalette.h. One thing I don't understand is why the minimum private data size for V_QUICKTIME has been the mysterioius value 21 so far, since 86 is the minimum size for a video sample description without a palette. So I changed that. As I said before, I have also changed the minimum private data size for A_QUICKTIME from 86 to 36, since 36 is the size of a version 0 sound sample description.
> Mats -- 
> Mats Peterson
> http://matsp888.no-ip.org/~mats/

>  matroskadec.c |   94 ++++++++++++++++++++++++++++++++++++++++++++++++++++++----
>  1 file changed, 89 insertions(+), 5 deletions(-)
> 6d1e32a75c7d8de7dfb02756a135ad7855c49b90  matroskadec.c.diff
> --- matroskadec.c.orig	2015-12-08 11:01:40.640478749 +0100
> +++ matroskadec.c	2015-12-11 12:00:13.000000000 +0100

this breaks demuxing of the video stream from
output-ffmpeg-20140109-git-c0a33c4.mkv
(google should find a url for this)


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151212/9436afff/attachment.sig>