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@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/
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to