From owner-freebsd-current Sat Dec 20 06:19:23 1997 Return-Path: Received: (from root@localhost) by hub.freebsd.org (8.8.7/8.8.7) id GAA19073 for current-outgoing; Sat, 20 Dec 1997 06:19:23 -0800 (PST) (envelope-from owner-freebsd-current) Received: from ns.mt.sri.com (sri-gw.MT.net [206.127.105.141]) by hub.freebsd.org (8.8.7/8.8.7) with ESMTP id GAA19060; Sat, 20 Dec 1997 06:19:14 -0800 (PST) (envelope-from nate@mt.sri.com) Received: from mt.sri.com (rocky.mt.sri.com [206.127.76.100]) by ns.mt.sri.com (8.8.8/8.8.8) with SMTP id HAA04455; Sat, 20 Dec 1997 07:19:10 -0700 (MST) (envelope-from nate@rocky.mt.sri.com) Received: by mt.sri.com (SMI-8.6/SMI-SVR4) id HAA24217; Sat, 20 Dec 1997 07:19:07 -0700 Date: Sat, 20 Dec 1997 07:19:07 -0700 Message-Id: <199712201419.HAA24217@mt.sri.com> From: Nate Williams MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit To: dyson@FreeBSD.ORG Cc: gurney_j@resnet.uoregon.edu, julian@whistle.com, bde@FreeBSD.ORG, current@FreeBSD.ORG Subject: Re: Bruce vandalism again In-Reply-To: <199712200904.EAA04490@dyson.iquest.net> References: <19971219215030.56082@hydrogen.nike.efn.org> <199712200904.EAA04490@dyson.iquest.net> X-Mailer: VM 6.29 under 19.15 XEmacs Lucid Sender: owner-freebsd-current@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk > John-Mark Gurney said: > > > I agree. This appears to be a seriously retrograde step. Bruce, why? > > > > easy, don't you guys read style(9)? > > Use of the __P macro in new code is discouraged, although modifications > > to existing files should be consistent with that file's conventions. > > > > all the other parts of the code in that file are K&R style... so you > > need to maintain that convention... > > > If style(9) is broken, which apparently it is, then it should be fixed. > It isn't a religious document, but one that should change with the times. I disagree. I'm with Bruce here in that you shouldn't 'bit by bit' change code to use new ANSI prototypes *UNLESS* you change all of the code to use ANSI prototypes. So, either convert them all to the new style (my preference), or leave them all the old style, but having a mix/match of both is silly. And, if you want to change it all to the new style, make the 'style' changes commits separate from the functional changes commits so one can see the functionality changes easy w/out having to dig through the style changes. Nate