Re: svn commit: rev 5449 - trunk/subversion/tests/clients/cmdline

Karl Fogel <kfogel@newton.ch.collab.net> writes:

> Ben Collins-Sussman <sussman@collab.net> writes:
> > 1. look for a sentence or paragraph that explains what the change is
> > about.
> >
> > 2. if still interested, read about the symbols changed.
> >
> > 3. if still interested, read the diffs.
>
> I should confess that I've written log message like Matt's before,
> too, so I hope I don't sound *too* preachy :-). However, I'm now
> trying to always write a summary blurb, the only exceptions being when
> there's just one place changed -- then the blurb and the technical
> description are one and the same.

The "what" of a change is important (and the change to the example log
message is an improvement), but once I'm at least casually familiar
with a body of code, I find the "why" a lot more important.

-- 
Daniel Rall <dlr@finemaltcoding.com>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org