Re: [patch] big whitespace cleanup in sys/kern/*

From: Thomas Dickey <dickey_at_radix.net>
Date: Tue, 3 Aug 2010 17:40:05 -0400
On Tue, Aug 03, 2010 at 11:22:36AM -0700, Julian Elischer wrote:
> On 8/3/10 2:34 AM, pluknet wrote:
> >Hi.
> >
> >I looked into sys/kern/* files to fix a bunch of common w/s style issues 
> >(221):
> >- leading space before label;
> >- leading space(s) before<tab>;
> >- space(s) instead of<tab>(s);
> >- space(s) in blank like.
> >
> >I tried to be conservative and didn't touch semi-contrib files and
> >those with its own style.
> >Here is a diff I'd like someone look into and check in if there will
> >no objections.
> 
> The style guide suggests against wholesale cleanups and we have 
> generally avoided them due to teh fact that they tend to obfuscate diffs.
> The idea being that we clean as we go..
> however it may be time for one.. I'd leave it to others to decide.

I'm curious why there's no mention of using 'indent' (with appropriate
settings...).

-- 
Thomas E. Dickey
http://invisible-island.net
ftp://invisible-island.net

Received on Tue Aug 03 2010 - 19:59:01 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:05 UTC