From owner-cvs-all Sat Jun 8 7:39:22 2002 Delivered-To: cvs-all@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id 241E337B406; Sat, 8 Jun 2002 07:39:14 -0700 (PDT) Date: Sat, 8 Jun 2002 07:39:14 -0700 From: "J. Mallett" To: Bruce Evans Cc: Bruce Evans , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/bin/ps extern.h Message-ID: <20020608073913.A89596@FreeBSD.ORG> References: <20020608000108.A42463@FreeBSD.ORG> <20020608225458.Q19691-100000@gamplex.bde.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20020608225458.Q19691-100000@gamplex.bde.org>; from bde@zeta.org.au on Sat, Jun 08, 2002 at 11:13:53PM +1000 Organisation: The FreeBSD Project X-Alternate-Addresses: , , , X-Affiliated-Projects: FreeBSD, xMach, ircd-hybrid-7 X-Towel: Yes 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 * From Bruce Evans > > I think sorting by type makes no sense for extern declarations. (style.9 > only says how to sort structs and local variables. It says to sort > by size [largest first] (not type), then by name. Sorting struct members > by size gives better packing, and sorting local variables by size > (perhaps smallest instead of largest first) may give gives better packing > for primitive compilers, but extern declarations don't allocate space so > size considerations don't apply.) Okay. > > we should be sorting as per LANG=C, and in that case, 'ra' definitely comes a > > bit after 'no', however 'no' comes happily after 'nl', therefore the lines in > > this locate could be written as: > > > > %%% > > extern int cflag, eval, fscale, mempages, nlistread; > > extern time_t now; > > extern int rawcpu, sumrusage, termwidth, totwidth; > > %%% > > > > But I don't think revision 1.24 fixes unsorting. > > I didn't want to restructure the declarations to have only one variable > per line, so I followed my usual rule for insertion "sorting" in unsorted > list: insert before the first line whose first item is after the item > being inserted. Since this gave a very trivial change, I should have > restructured the declarations. Ah ha, okay. Thanks. -- J. Mallett FreeBSD: The Power To Serve "I've coined new words, like, misunderstanding and Hispanically." -- George W. Bush, Radio-Television Correspondents Association dinner, Washington, D.C., March 29, 2001 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message