[FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

compn tempn at mi.rr.com
Fri Dec 11 15:08:53 CET 2015


On Fri, 11 Dec 2015 12:59:16 +0000 (UTC)
Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> wrote:

> I just want someone of you guys to take a look at what I've done. You
> know Git much better than me, and you can create a pull request it
> looks decent enough. I'm not going to participate a lot in this in
> the future, but I wanted to correct these things, since they have
> been overlooked for such a long time. Mats -- Mats Peterson

thanks for contributing. 

it can be daunting for new contributors to follow all of our arcane
rules.

anyone can split up patches, just using diff -u and a couple new files.
doesnt have to be done with git.

you've done your job, the patch is submitted. 

a question about changing the a_quicktime limit is did you test a bunch
of other a_quicktime samples to make sure nothing broke?  thats always
fun when changing things is that we have to retest all other samples we
have from http://samples.ffmpeg.org . not sure we even have that many
a_quicktime in mkv samples. same with v_quicktime.

-compn


More information about the ffmpeg-devel mailing list