From owner-cvs-all Wed Sep 5 3:40:15 2001 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 972D637B408; Wed, 5 Sep 2001 03:40:05 -0700 (PDT) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id UAA27903; Wed, 5 Sep 2001 20:40:01 +1000 Date: Wed, 5 Sep 2001 20:39:34 +1000 (EST) From: Bruce Evans X-X-Sender: To: "David O'Brien" Cc: Garrett Wollman , , Subject: Re: cvs commit: src/include/arpa tftp.h In-Reply-To: <20010904185643.A78975@dragon.nuxi.com> Message-ID: <20010905203608.K22645-100000@alphplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, 4 Sep 2001, David O'Brien wrote: > On Tue, Sep 04, 2001 at 09:38:36PM -0400, Garrett Wollman wrote: > > < said: > > > > > include/arpa tftp.h > > > Log: > > > style(9) the structure definitions. > > > > Was this *really* necessary? > > > > I thought we had a policy against doing broad sweeps of > > non-substantive changes like this. > > Yes it was -- we were so confused (partically do to your commit to > style.9) that one could not figure out what was right. Now enough of the > key places to look for style guidlines have been fixed. I don't plan on > sweaping any more of the tree to just fix this. The existence of both in old code means that they were both right. "struct\t.*{" does seem to be the least normal form although it is used in the oldest code (proc.h...), but you shouldn't have changed it while we are still discussing this. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message