[FFmpeg-devel] [PATCH] Some minor fixes of my "patchset" for palettized QuickTime video in Matroska

Ganesh Ajjanagadde gajjanag at mit.edu
Mon Dec 21 17:30:26 CET 2015


On Mon, Dec 21, 2015 at 8:25 AM, Clément Bœsch <u at pkh.me> wrote:
> On Mon, Dec 21, 2015 at 05:16:36PM +0100, Mats Peterson wrote:
> [...]
>> As if that wasn't enough, here is another update. This time it's passing the
>> FATE tests, unlike the old one. In order to refresh your memories, this
>> patch fixes the long-standing issue regarding not recognizing palettized
>> QuickTime video in Matroska. I have created a new function get_qtpalette()
>> in a new file qtpalette.c, that is shared by both matroskadec.c and mov.c.
>> And just like before, I'm not really in the mood to learn Git just to
>> contribute with these simple patches, but I would be happy if someone could
>> take a look at it, and, if it looks decent enough, submit proper patches in
>> the right format for Git.
>>
>
> No one will open your zip. The problem with the tar wasn't the archive
> format but the fact that it was an archive.
>
> Also, if you don't submit a proper patch (it's 2 git command¹, nothing
> crazy), you will get ignored like the dozen of mails you already sent.
>
> Regards,
>
> [1]: git commit -a -m "lavf/matroska: ..." && git format-patch -1

To add to Clement's message, Lou has already given you a complete git tutorial.

>
> --
> lément B.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list