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

[Mats Peterson]

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

-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/