Patch reviewing: style vs. content [was: [PATCH] Issue 1628]

Mark Phippard wrote (in an email entitled "Re: [PATCH] Issue 1628"):
>
> [...] I think if you are inclined to
> review a patch the first thing you should ask yourself is whether you have
> all of the "meta-information" you need to do so. Does the log message or
> the email include enough of this information so that you are clear about
> and agree with the objectives? [...]

Excellent advice. Thanks.

But doesn't that meta-information including having a relevant Subject line on
the email? :-) Hint, hint, nag, nag! Have you noticed that's one of my pet
peeves?

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 25 19:47:40 2005