[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