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] |
On Wed, 2003-07-23 at 02:29, Max Bowsher wrote: > May I apply whitespace changes which are in keeping with the style of > surrounding code and the rest of setup without explicit review? > > Are such things supposed to be ChangeLogged? > > Example: > -UINT Window::IsButtonChecked (int nIDButton) const > +UINT > +Window::IsButtonChecked (int nIDButton) const http://www.cygwin.com/ml/cygwin-apps/2003-03/msg00750.html IOW, no, no review for whitespace, but note that I'll likely be running astyle (a code formatter) without notice when I get some time... Comments that are on their own - isolated changes - should be changelogged though, as should non-autogenerated documentation. Why? Because doco changes also have motive - like code changes, and like code changes the motive should be recorded. Rob Rob -- 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] |