[FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

Mats Peterson matsp888 at yahoo.com
Wed Dec 16 18:13:01 CET 2015

On 12/16/2015 06:07 PM, Mats Peterson wrote:
> On 12/15/2015 07:25 PM, Mats Peterson wrote:
>> On 12/15/2015 06:51 PM, Mats Peterson wrote:
>>> Here is a *suggestion* for how to end the QuickTime palette in Matroska
>>> saga. I have created a new file 'qtpalette.c' with a function
>>> get_qtpalette(), that will be called from both matroskadec.c and mov.c.
>>> So, no more calling into another demuxer to get the palette, and no
>>> duplicated code, since it is more or less moved from mov.c to this new
>>> file. The mov_parse_stsd_video() function in mov.c is rather heavily
>>> modified as a consequence. Now, please forgive me for not splitting it
>>> up, but I hope that someone will be able to try it out in spite of this.
>>> Mats
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel at ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> Once again, test files with palettized video, both QuickTime and
>> Matroska, can be found at
>> https://drive.google.com/open?id=0B3_pEBoLs0faWElmM2FnLTZYNlk
>> Mats
> I should add that I know squat about the workings of Git, and I don't
> have time or enough motivation to learn all the complexities of it, so I
> would be grateful if anyone of you perhaps some time could create a set
> of proper patches out of what I've done in this archive, if it looks
> sensible enough to you. Thanks in advance.
> Mats

I forgot to remove a "fprintf(stderr, "COLOR COUNT: %d\n", 
color_count);" line in qtpalette.c, but it's easy to find.


Mats Peterson

More information about the ffmpeg-devel mailing list