[FFmpeg-devel] [PATCH] matroskadec.c: New patch for palettized QuickTime video (old one broken)

Mats Peterson matsp888 at yahoo.com
Sun Dec 13 21:16:00 CET 2015

On 12/13/2015 09:05 PM, Mats Peterson wrote:
> On 12/13/2015 09:02 PM, Mats Peterson wrote:
>> On 12/13/2015 08:48 PM, Mats Peterson wrote:
>>> I just discovered that I've been incorrectly using the "i" loop variable
>>> in my patch, which is actually used by a track element loop at line 1718
>>> in matroskadec.c. This will works as long as there is only video in the
>>> Matroska file, that's why I didn't discover it until I tried one with
>>> audio as well (all the samples so far are video only).
>>> Mats
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel at ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> I forgot to restore the lower limit of A_QUICKTIME private data to 21
>> like Michael Niedermayer proposed. Sorry (it doesn't make much sense to
>> me, but at least that broken file will pass.)
>> Mats
> V_QUICKTIME (video) of course, not A_QUICKTIME.
> Mats

Well, probably it will NOT pass, since I'm looking for the bit depth at 
offset 82 of the private data, which will of course be a totally bogus 
value with a private data size of 21. Oh well...


Mats Peterson

More information about the ffmpeg-devel mailing list