From: Junio C Hamano Date: Mon, 12 Dec 2011 23:41:39 GMT Subject: Re: [PATCH v2] Update documentation for stripspace Message-ID: <7vwra1z7bg.fsf@alter.siamese.dyndns.org> In-Reply-To: <1323728909-7847-1-git-send-email-conrad.irwin@gmail.com> Conrad Irwin writes: > ... > I've moved the *NOTE* into a SEE ALSO section where I think it reads > less opinionatedly ― is that better? I think it looks a lot worse. At least your original hinted that some people may confuse the two and the NOTE was there for such people; other people who would not even dream of such a confusion won't find the existence of the note a "Huh?". But the updated patch with a link in SEE ALSO section without any explanation would be a definite "Huh?" for those of us who find that stripspace does not have anything to do with what "apply --whitespace=fix" does. The new example section looks good. Perhaps we can just drop the extra SEE ALSO and resurrect the *NOTE* from your v1 patch. Thanks.