From owner-freebsd-arch Thu Jan 31 0: 9: 9 2002 Delivered-To: freebsd-arch@freebsd.org Received: from softweyr.com (softweyr.com [65.88.244.127]) by hub.freebsd.org (Postfix) with ESMTP id A9CA137B416 for ; Thu, 31 Jan 2002 00:09:04 -0800 (PST) Received: from homer.softweyr.com ([204.68.178.39] helo=softweyr.com) by softweyr.com with esmtp (Exim 3.33 #1) id 16WCH2-000319-00; Thu, 31 Jan 2002 01:08:48 -0700 Message-ID: <3C58FBEC.746257BB@softweyr.com> Date: Thu, 31 Jan 2002 01:10:20 -0700 From: Wes Peters Organization: Softweyr LLC X-Mailer: Mozilla 4.78 [en] (X11; U; Linux 2.4.2 i386) X-Accept-Language: en MIME-Version: 1.0 To: Terry Lambert Cc: Jeroen Ruigrok/asmodai , Kirk McKusick , arch@FreeBSD.ORG, Peter Wemm , Poul-Henning Kamp , Dallas De Atley , Jordan Hubbard , "Perry E. Metzger" , "Todd C. Miller" , Theo de Raadt Subject: Re: __P macro question References: <68578.1012450924@winston.freebsd.org> <200201310701.g0V71Ci75803@beastie.mckusick.com> <20020131072933.GQ22384@daemon.ninth-circle.org> <3C58F78E.3F66EA8E@mindspring.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Terry Lambert wrote: > > Jeroen Ruigrok/asmodai wrote: > > Which items can we identify to tackle in this effort? > > > > - Get rid of __P() macros in source files > > - Use proper ANSI prototypes, this flows from the point above Sort of. If you mean "edit the __P() macros into prototypes with the same footprint", that's a good starting point. A logical next step would be to check each of our newly prototyped functions against Posix/SUS and become more standards compliant. > > what else? > > Other candidates are other macros in cdefs.h; because of the > __attribute stuff, I would be loathe to get rid of it entirely, > since it encapsulates some GCC dependence/independence, but the > const/void/volatile definitions are a possibility. > > There are also the varradic function declarations, which are > not all ANSI-C style, yet, use of __STRING and __CONCAT, etc.. > > My recommendation would be to do __P() first, now that there > is a cross-BSD consensus, and leave other changes for other > discussions. Always a sensible choice, just to keep the CVS commits atomic if no other reason. -- "Where am I, and what am I doing in this handbasket?" Wes Peters Softweyr LLC wes@softweyr.com http://softweyr.com/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message