Date: Tue, 3 Aug 2010 17:21:45 -0700 From: Steve Kargl <sgk@troutmask.apl.washington.edu> To: Thomas Dickey <dickey@radix.net> Cc: FreeBSD Current <freebsd-current@freebsd.org>, Julian Elischer <julian@elischer.org> Subject: Re: [patch] big whitespace cleanup in sys/kern/* Message-ID: <20100804002145.GA76309@troutmask.apl.washington.edu> In-Reply-To: <20100803214005.GA539@saltmine.radix.net> References: <AANLkTi=E%2Bzg5rNSv0Dg6uG%2Baxxvz=vrRVdug0JiNhbHG@mail.gmail.com> <4C585E6C.7080006@elischer.org> <20100803214005.GA539@saltmine.radix.net>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100804002145.GA76309>