[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