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

Diego Biurrun diego
Tue Mar 3 18:16:02 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 descriptive log messages; most log messages around here are
very short and often lack sufficient detail.  I agree that the first line
should offer a quick synopsis of the change though.

Diego




More information about the ffmpeg-cvslog mailing list