On Mon, Dec 21, 2015 at 9:49 AM, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote: > On 12/21/2015 06:04 PM, Mats Peterson wrote: >> >> Alright then, let's see how this turns out: >> > > Perhaps I should once again explain the issue: > > Palettized QuickTime video in Matroska has hitherto not been recognized > whatsoever, and the "palette" used has been completely random. > > The patch for matroskadec.c fixes this issue by adding a palette side data > packet in matroska_deliver_packet(), much in the same way as it's done in > mov.c. > > The change to mov.c consists mainly of moving the palette handling from the > mov_parse_stsd_video() function to a new get_qtpalette() function in the new > file qtpalette.c, which is shared by both matroskadec.c and mov.c. > > In matroskadec.c, I'm also putting the palette in 'extradata', like it's > done for V_MS/VFW/FOURCC; this is a requirement in order for MPlayer to > recognize the palette.
Please add such things to the commit message, i.e while doing git commit, it fires up your editor. Write the rationale in the same place you wrote: "Alright then, let's see how this turns out:". > > Mats > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
