[FFmpeg-devel] [PATCH] libavformat/matroskadec.c, mov.c/qtpalette.c: Fix issue with, palettized QuickTime video in Matroska

Ganesh Ajjanagadde gajjanag at mit.edu
Mon Dec 21 19:02:37 CET 2015


On Mon, Dec 21, 2015 at 9:49 AM, Mats Peterson
<matsp888-at-yahoo.com at 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 at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list