From owner-cvs-all@FreeBSD.ORG Mon Aug 16 03:16:08 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 031CE16A4CE; Mon, 16 Aug 2004 03:16:08 +0000 (GMT) Received: from ebb.errno.com (ebb.errno.com [66.127.85.87]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9AB6043D48; Mon, 16 Aug 2004 03:16:07 +0000 (GMT) (envelope-from sam@errno.com) Received: from [66.127.85.91] ([66.127.85.91]) (authenticated bits=0) by ebb.errno.com (8.12.9/8.12.6) with ESMTP id i7G3G7Wi063096 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NO); Sun, 15 Aug 2004 20:16:07 -0700 (PDT) (envelope-from sam@errno.com) From: Sam Leffler Organization: Errno Consulting To: Garance A Drosihn Date: Sun, 15 Aug 2004 20:18:34 -0700 User-Agent: KMail/1.6.1 References: <200408150624.i7F6OhhR074096@repoman.freebsd.org> <20040816023851.GC3026@green.homeunix.org> In-Reply-To: MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200408152018.35018.sam@errno.com> cc: Brian Fundakowski Feldman cc: John-Mark Gurney cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: cvs-src@FreeBSD.org Subject: Re: cvs commit: src/sys/cam/scsi scsi_target.c src/sys/dev/mii mii.c src/sys/fs/fifofs fifo_vnops.c src/sys/gnu/ext2fs ext2_vnops.c src/sys/kern init_main.c kern_conf.c kern_descrip.c kern_event.c kern_exec.c kern_exit.c kern_fork.c kern_sig.c sys_pipe.c tty.c ... X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Aug 2004 03:16:08 -0000 On Sunday 15 August 2004 08:00 pm, Garance A Drosihn wrote: > At 10:38 PM -0400 8/15/04, Brian Fundakowski Feldman wrote: > >On Sun, Aug 15, 2004 at 06:51:08PM -0700, John-Mark Gurney wrote: > > > sure, I'd like a quick peak at the patch though (if it takes me > > > more than a day, go ahead and commit). > > > >I'd very much like you to look it over. I haven't tested it, just > >made the stylistic changes. If it were more complete (i.e. satisfy > >bde), every spurious blank line (that is, all of them inside functions > >which are not between variable declarations and code) would be gone, > >too, but that can be kind of harsh. > > > >--- /usr/src/sys/kern/kern_event.c Sun Aug 15 15:44:41 2004 > >+++ kern_event.c Sun Aug 15 22:36:25 2004 > > > >@@ -232,7 +231,6 @@ > > static int > > filt_fileattach(struct knote *kn) > > { > >- > > return (fo_kqfilter(kn->kn_fp, kn)); > > } > > In some cases, such as this one, you are removing a blank line > which style(9) wants to have there. ISTR being told that if there > are no local variables in a routine, there should be a blank line > before the first executable statement. Looking at the man page, I > think that is shown by the example: > > static void > usage() > { > /* Insert an empty line if the function has no local variables. */ Repeat after me: "style is a guideline, not a contract." Having grossly inconsistent style is bad. The only ironclad rule I believe should be enforced is "use consistent style when modifying existing code." Everything else should be up to the discretion of the programmer and people that come through after and make gratuitous style changes are not being helpful. And in case it's not clear I am _not_ picking on you or your comments. Sam Sam