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

From: Steve Kargl <sgk_at_troutmask.apl.washington.edu>
Date: Tue, 3 Aug 2010 17:21:45 -0700
On Tue, Aug 03, 2010 at 05:40:05PM -0400, Thomas Dickey wrote:
> 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...).
> 

Because there is no set of appropriate settings for indent(1)
to reproduce style(9).

-- 
Steve
Received on Tue Aug 03 2010 - 22:21:48 UTC

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