Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 3 Aug 2010 17:40:05 -0400
From:      Thomas Dickey <dickey@radix.net>
To:        FreeBSD Current <freebsd-current@freebsd.org>
Cc:        Julian Elischer <julian@elischer.org>
Subject:   Re: [patch] big whitespace cleanup in sys/kern/*
Message-ID:  <20100803214005.GA539@saltmine.radix.net>
In-Reply-To: <4C585E6C.7080006@elischer.org>
References:  <AANLkTi=E%2Bzg5rNSv0Dg6uG%2Baxxvz=vrRVdug0JiNhbHG@mail.gmail.com> <4C585E6C.7080006@elischer.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--NzB8fVQJ5HfG6fxh
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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=
=20
> >(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.
>=20
> The style guide suggests against wholesale cleanups and we have=20
> 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...).

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

--NzB8fVQJ5HfG6fxh
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (SunOS)
Comment: For info see http://www.gnupg.org

iD8DBQFMWIyztIqByHxlDocRAk2dAKCL4+kRycAqdb9wmRPRxe6J4aBO+gCeKphq
LTW8ZyNAQpUgfJUbtumkxEw=
=HV9w
-----END PGP SIGNATURE-----

--NzB8fVQJ5HfG6fxh--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100803214005.GA539>