[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