From owner-freebsd-audit Wed Jul 17 2:16: 1 2002 Delivered-To: freebsd-audit@freebsd.org Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id AD12337B400 for ; Wed, 17 Jul 2002 02:15:58 -0700 (PDT) Received: from storm.FreeBSD.org.uk (storm.FreeBSD.org.uk [194.242.139.170]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9D45A43E31 for ; Wed, 17 Jul 2002 02:15:53 -0700 (PDT) (envelope-from mark@grimreaper.grondar.org) Received: from storm.FreeBSD.org.uk (uucp@localhost [127.0.0.1]) by storm.FreeBSD.org.uk (8.12.5/8.12.5) with ESMTP id g6H9Fh9G037137; Wed, 17 Jul 2002 10:15:44 +0100 (BST) (envelope-from mark@grimreaper.grondar.org) Received: (from uucp@localhost) by storm.FreeBSD.org.uk (8.12.5/8.12.5/Submit) with UUCP id g6H9FggN037136; Wed, 17 Jul 2002 10:15:42 +0100 (BST) Received: from grimreaper.grondar.org (localhost [127.0.0.1]) by grimreaper.grondar.org (8.12.5/8.12.5) with ESMTP id g6H9F2uH037129; Wed, 17 Jul 2002 10:15:02 +0100 (BST) (envelope-from mark@grimreaper.grondar.org) Message-Id: <200207170915.g6H9F2uH037129@grimreaper.grondar.org> To: Bruce Evans Cc: freebsd-audit@FreeBSD.ORG Subject: Re: bin/ln & WARNS=5 References: <20020717081218.N2535-100000@gamplex.bde.org> In-Reply-To: <20020717081218.N2535-100000@gamplex.bde.org> ; from Bruce Evans "Wed, 17 Jul 2002 08:26:34 +1000." Date: Wed, 17 Jul 2002 10:15:02 +0100 From: Mark Murray Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG > > > +Some code in the tree does not follow the last part of this rule. > > > +Do not change it gratuitously to do so. > > > > Why not? We gratuitously remove register. The above is a rule with > > no reason. > > Because it gives myriads of gratuitous changes. There should be a rule > about not gratuitously removing register too. Actually, we already have > one near the end of the file: > > % Stylistic changes (including whitespace changes) are hard on the source > % repository and are to be avoided without good reason. This then comes down to what is defined as "good reason". I believe that cleaning up code, fixing warnings and generally modernising ugly code is good reason. > My version of style.9 also has this meta-rule near the start of the file: > > % /* > % * Internal consistency (per-file) has precedence over these rules. > % */ > % .Ed > % .\" Most files are not internally consistent, so these rules usually > % .\" apply :-). Sort of. take a look at some very old Sun code - rpcgen is a good one. It siffers from a bad case of int-itis (everything fits into an int or defaults to an int). Maintaining stagnant code for purely repo bloat reasons is a dumb rule, I think. The SCM system is to allow forward movement, not to hold us back. In patchkit days, such changes would have been insane. Now we have the tools, and constructing reasons not to use them is to lose the plot. M -- o Mark Murray \_ O.\_ Warning: this .sig is umop ap!sdn To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message