"diff" vs "for review"

Fergus Henderson fjh at cs.mu.OZ.AU
Thu Mar 25 15:24:36 AEDT 1999


On 25-Mar-1999, Peter Ross <petdr at cs.mu.OZ.AU> wrote:
> If anyone wants to review this feel free.  I have already committed the
> changes.

Normally we use "diff:" rather than "for review:" if the change has
already been committed.  I think the intended meanings of the
various prefixes is roughly as follows:

	prefix		meaning
	------		-------
	for review:	Please review this.  Not yet committed.
			There's a good chance that this change
			will be modified in response to reviewers'
			comments.

	diff:		Already committed, but a review is probably
			a good idea anyway.

	trivial diff:	Already committed, and the change is trivial,
			so no review should be needed.
			Read this message only if you are paranoid.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh>  |  of excellence is a lethal habit"
PGP: finger fjh at 128.250.37.3        |     -- the last words of T. S. Garp.



More information about the developers mailing list