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?


Christopher Faylor wrote:
> On Tue, Jul 22, 2003 at 05:29:45PM +0100, 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?
>
> Do we have an obvious checkin rule for setup?

No, not yet. This is the first time I've had multiple obvious changes to
make.

> This is obviously obvious.

Yes, that much is obvious :-)

>> Are such things supposed to be ChangeLogged?
>
> I think opinions on this are mixed but I usually don't add one for white
> space or comment changes since they should be very self-explanatory.

OK.


Max.


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