From owner-freebsd-audit Wed Jul 3 10: 0:11 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 6D0DA37B400 for ; Wed, 3 Jul 2002 10:00:04 -0700 (PDT) Received: from dragon.nuxi.com (trang.nuxi.com [66.92.13.169]) by mx1.FreeBSD.org (Postfix) with ESMTP id CF4A043E42 for ; Wed, 3 Jul 2002 10:00:03 -0700 (PDT) (envelope-from obrien@NUXI.com) Received: from dragon.nuxi.com (obrien@localhost [127.0.0.1]) by dragon.nuxi.com (8.12.5/8.12.2) with ESMTP id g63H03Xu008663; Wed, 3 Jul 2002 10:00:03 -0700 (PDT) (envelope-from obrien@dragon.nuxi.com) Received: (from obrien@localhost) by dragon.nuxi.com (8.12.5/8.12.5/Submit) id g63Gxvos008640; Wed, 3 Jul 2002 09:59:57 -0700 (PDT) Date: Wed, 3 Jul 2002 09:59:56 -0700 From: "David O'Brien" To: Mark Murray Cc: audit@FreeBSD.ORG Subject: Re: lib/csu cleanup #2 - review please (commit candidate) Message-ID: <20020703095956.A8415@dragon.nuxi.com> Reply-To: audit@FreeBSD.ORG Mail-Followup-To: David O'Brien , Mark Murray , audit@FreeBSD.ORG References: <200206231830.g5NIUAa4045990@grimreaper.grondar.org> <200206301924.g5UJOL07070228@grimreaper.grondar.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <200206301924.g5UJOL07070228@grimreaper.grondar.org>; from mark@grondar.za on Sun, Jun 30, 2002 at 08:24:21PM +0100 X-Operating-System: FreeBSD 5.0-CURRENT Organization: The NUXI BSD group X-Pgp-Rsa-Fingerprint: B7 4D 3E E9 11 39 5F A3 90 76 5D 69 58 D9 98 7A X-Pgp-Rsa-Keyid: 1024/34F9F9D5 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 On Sun, Jun 30, 2002 at 08:24:21PM +0100, Mark Murray wrote: > In response to some useful comments from Bruce and David, I've > cleaned this up, tested it on i386-elf (on my laptop) and extended > it to the other architectures a lot more (limited testing, so > please help out!). But you didn't answer these questions: > Index: alpha/crt1.c > =================================================================== > RCS file: /home/ncvs/src/lib/csu/alpha/crt1.c,v ... > +#ifndef lint > + > #ifndef __GNUC__ > #error "GCC is needed to compile this file" > #endif Please change Lint to note the #error but to continue processing. I've pondered it and cannot find a good reason for Lint to "obey" #error. > -_start(char *arguments, ...) > +_start(char *ap, ...) Why do we need to rename "arguments"? > +extern int _DYNAMIC; > +#pragma weak _DYNAMIC Why? IMO #pragma's should come as early as possible. (#also sorts before `e'xtern) New questions: > -_start(char **ap, > - void (*cleanup)(void), /* from shared loader */ > - struct Struct_Obj_Entry *obj __unused, /* from shared loader */ > - struct ps_strings *ps_strings __unused) > +_start(char **ap, void (*cleanup)(void), ...) Why take a more exact prototype and definition to a lesser one? > - fptr rtld_cleanup; > + fptr cleanup; ... > - rtld_cleanup = get_rtld_cleanup(); > + cleanup = get_rtld_cleanup(); Why do we need to rename this? (you also have a ^I above) > diffs in i386-elf/crt1.c (Which I will commit separately; it > has 4-space indents instead of tabs, and I'd like to fix that). Why? This is not abandoned code. Our practice is to leave existing style alone when making small changes. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message