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

Mats Peterson matsp888 at yahoo.com
Fri Dec 11 13:59:16 CET 2015


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
http://matsp888.no-ip.org/~mats/
      From: Moritz Barsnick <barsnick at gmx.net>
 To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org> 
 Sent: Friday, December 11, 2015 1:55 PM
 Subject: Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly
   
On Fri, Dec 11, 2015 at 11:48:15 +0000, Mats Peterson wrote:
> And I said before, I'm not going to "learn" how to use the messy Git
> system.

What's messy?

That's just the way it works here. It may be a bit more tedious for
you, but makes life for everyone else (testers, reviewers, committers)
much more easy. Complex projects need more elaborate flows.

And even in any other serious project, diffs directly from within the
leaf source directory are frowned upon. Create the diff from one level
up from the top (so that it applies from the top level within the tree
with "patch -p1"). Everything else needs too much contemplation.

I can't comment on the rest.

Moritz


_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel at ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


 


More information about the ffmpeg-devel mailing list