[FFmpeg-cvslog] r23977 - trunk/libavcodec/rl2.c

Reimar Döffinger Reimar.Doeffinger
Thu Jul 8 21:21:27 CEST 2010


On Thu, Jul 08, 2010 at 08:12:02PM +0100, M?ns Rullg?rd wrote:
> Diego, and everybody else here, is doing his best to help improve
> things.  The rest of us here have jobs and/or school to share our time
> with, and try to get the most out of the time we put into FFmpeg.  All
> too often you have rejected patches for no other reason than not being
> provably perfect, even though they were without question an
> improvement.

Well, I do think that more often a "definitely better than what we have"
would be a good reason to ok a patch on its own.
However I was a bit annoyed about parts of the patch as well, because it
seemed remove documentation. Even when it does not apply 100% anymore,
just removing it in my opinion is far from an improvement, and would have
warrented at least asking about it before....



More information about the ffmpeg-cvslog mailing list