[FFmpeg-cvslog] r17767 - in trunk/libavformat: rtp_h264.c rtpdec.c

Reimar Döffinger Reimar.Doeffinger
Tue Mar 3 17:55:23 CET 2009


On Tue, Mar 03, 2009 at 11:27:41AM -0500, Ronald S. Bultje wrote:
> On Tue, Mar 3, 2009 at 9:23 AM, Reimar D?ffinger
> <Reimar.Doeffinger at gmx.de> wrote:
> > The reference is good, but IMO you should have used it to reduce the
> > commit message to something someone only marginally interested would
> > read. Personally I consider it far too long to read and as a consequence
> > I don't really have any idea what the point of the change is (the first
> > 3 sentences don't really clarify it).
> 
> If there's general consensus, I'll reduce my log message size. I
> always intentionally kept them long so that at later points, it would
> be relatively easy to see why I made this particular change.

I appreciate the idea, I just think that even then you won't be happy to
have to read so much (you will probably remember it mostly and just need
a short reminder, not such a long text).
IMHO if you write such a long message that is ok (for me), but make sure
that the basic idea behind it is clear after reading at most the first 3 lines.




More information about the ffmpeg-cvslog mailing list