This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Review required for whitespace changes?


On Wed, 2003-07-23 at 11:46, Christopher Faylor wrote:


> I'd submit that if the motivation for a comment change isn't obvious
> then the comment change either shouldn't have been made or the comment
> isn't clear.

I agree. Thats why I qualified the rule-of-thumb: comment changes as
part of a larger change - no need to mention. comment changes on their
own though, I'd like to know /why/ they needed to change. (Other than
the blindingly obvious fixup-a-spelling-error).

-- 
GPG key available at: <http://members.aardvark.net.au/lifeless/keys.txt>.

Attachment: signature.asc
Description: This is a digitally signed message part


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]