[FFmpeg-cvslog] r14611 - trunk/libavformat/matroskadec.c

Aurelien Jacobs aurel
Tue Aug 5 19:48:12 CEST 2008


Reimar Doeffinger wrote:

> On Tue, Aug 05, 2008 at 02:43:02AM +0200, aurel wrote:
> > Author: aurel
> > Date: Tue Aug  5 02:43:01 2008
> > New Revision: 14611
> > 
> > Log:
> > matroskadec: remove useless num_streams
> 
> I'd like to say that I am very unhappy with this kind of commit-flooding and would very
> strongly prefer if you would start committing you changes a bit earlier and not only after
> it has become such a massive amount...

I also wasn't very happy about this commit flood.
But I had no better solution.

In fact I started those changes as one big patch which quickly
became un-reviewable. Then I split it in many patches. And then
I added a few more patches which required me to modify some of
the very first patches (thanks to stgit).
So the only real alternative here would have been one huge
un-reviewable commit.

So trust me that I don't like such flood and that I would have
avoided it if there were a reasonable way to do so.

Aurel




More information about the ffmpeg-cvslog mailing list