Date: Fri, 14 Feb 2003 13:47:48 -0800 From: David Schultz <dschultz@uclink.Berkeley.EDU> To: Alfred Perlstein <alfred@FreeBSD.ORG> Cc: Sam Leffler <sam@errno.com>, src-committers@FreeBSD.ORG, cvs-src@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/kern kern_shutdown.c Message-ID: <20030214214748.GA2520@HAL9000.homeunix.com> In-Reply-To: <20030214191957.GO93252@elvis.mu.org> References: <200302141244.h1ECimG1059062@repoman.freebsd.org> <275601c2d43b$ebd9e580$52557f42@errno.com> <20030214191957.GO93252@elvis.mu.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thus spake Alfred Perlstein <alfred@FreeBSD.ORG>: > * Sam Leffler <sam@errno.com> [030214 07:15] wrote: > > > alfred 2003/02/14 04:44:48 PST > > > > > > Modified files: > > > sys/kern kern_shutdown.c > > > Log: > > > style. > > > > > > Revision Changes Path > > > 1.141 +20 -8 src/sys/kern/kern_shutdown.c > > > > > http://cvsweb.FreeBSD.org/src/sys/kern/kern_shutdown.c.diff?r1=1.140&r2=1.14 > > 1 > > > > More like "gratuitous style". > > More like when looking at the file in otder to fix other things I was so > turned off by it that it offended me and for that reason needed fixing. A single extra space at the end of a line offends you? Some of the ``style problems'' fixed by this commit aren't even in style(9). However, the following IS in style(9): Stylistic changes (including whitespace changes) are hard on the source repository and are to be avoided without good reason. People don't seem to understand that violating this principle makes it very hard to maintain local patches. If the file is being modified heavily anyway and there are major formatting bugs, then there's nothing wrong with a few style fixes, but people need to keep in mind that they *are* creating a hassle for others. Don't take this personally, even though I'm using this commit as an example; this is a generic complaint of mine against a practice, not meant as an attack on any specific individuals. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030214214748.GA2520>