From owner-freebsd-sparc64@FreeBSD.ORG Wed Dec 29 08:09:24 2010 Return-Path: Delivered-To: freebsd-sparc64@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 38F7C106564A for ; Wed, 29 Dec 2010 08:09:24 +0000 (UTC) (envelope-from gahr@gahr.ch) Received: from cpanel05.rubas-s05.net (cpanel05.rubas-s05.net [195.182.222.75]) by mx1.freebsd.org (Postfix) with ESMTP id BA6E48FC17 for ; Wed, 29 Dec 2010 08:09:23 +0000 (UTC) Received: from 4-221.198-178.cust.bluewin.ch ([178.198.221.4] helo=gahrfit.gahr.ch) by cpanel05.rubas-s05.net with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.69) (envelope-from ) id 1PXr61-0004Zc-9k; Wed, 29 Dec 2010 09:09:22 +0100 Received: by gahrfit.gahr.ch (sSMTP sendmail emulation); Wed, 29 Dec 2010 09:05:29 +0100 From: "Pietro Cerutti" Date: Wed, 29 Dec 2010 09:05:29 +0100 To: Marius Strobl Message-ID: <20101229080528.GB72669@gahrfit.gahr.ch> References: <20101228132703.GC68770@gahrfit.gahr.ch> <20101228173529.GA38083@alchemy.franken.de> <201012281310.50739.jhb@freebsd.org> <20101228182049.GV61199@alchemy.franken.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="T4sUOijqQbZv57TR" Content-Disposition: inline In-Reply-To: <20101228182049.GV61199@alchemy.franken.de> X-PGP-Key: 0x9571F78E X-PGP-Fingerprint: 1203 92B5 3919 AF84 9B97 28D6 C0C2 6A98 9571 F78E User-Agent: Mutt/1.5.21 (2010-09-15) X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - cpanel05.rubas-s05.net X-AntiAbuse: Original Domain - freebsd.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - gahr.ch X-Source: X-Source-Args: X-Source-Dir: Cc: freebsd-sparc64@freebsd.org Subject: Re: [patch] rename macro parameter list in cpufunc.h X-BeenThere: freebsd-sparc64@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: gahr@FreeBSD.org List-Id: Porting FreeBSD to the Sparc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 29 Dec 2010 08:09:24 -0000 --T4sUOijqQbZv57TR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2010-Dec-28, 19:20, Marius Strobl wrote: > On Tue, Dec 28, 2010 at 01:10:50PM -0500, John Baldwin wrote: > > On Tuesday, December 28, 2010 12:35:29 pm Marius Strobl wrote: > > > On Tue, Dec 28, 2010 at 02:27:04PM +0100, Pietro Cerutti wrote: > > > > Dear all, > > > >=20 > > > > sys/sparc64/include/cpufunc.h defines macro having a parameter named > > > > 'xor'. This is a reserved keyword in C++. For this reason, it is > > > > impossible to use this header in C++ (e.g., by including > > > > machine/atomic.h). > > > >=20 > > > > This patch [1] renames these parameters to _xor. Please review and/= or > > > > approve (I need an explicit approval in order to commit to src). > > > >=20 > > > > Thanks! > > > >=20 > > > > Best Regards, > > > >=20 > > > > [1] http://people.freebsd.org/~gahr/cpufunc.h.diff > > >=20 > > > Hrm, I really don't like the inconsistency only renaming "xor" > > > introduces, rototilling the whole file also seems excessive though > > > (also that would get it closer to style(9)). Would wrapping it in > > > __BEGIN_DECLS or something like that work as an alternate solution? > >=20 > > Not for a reserved word like 'class'. Maybe you could rename 'xor' to= =20 > > something else like 'mask' instead? > >=20 >=20 > The description of the assembly syntax uses "xor" so I'd prefer to keep > the name of the parameter close to that, I'd be okay with using f.e. > "xorval" though. Well, I guess the choice of the name is up to you, and xorval kind of makes sense to me. Would you like to commit it or I do? --=20 Pietro Cerutti The FreeBSD Project gahr@FreeBSD.org PGP Public Key: http://gahr.ch/pgp --T4sUOijqQbZv57TR Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEARECAAYFAk0a68cACgkQwMJqmJVx944PFgCg08Eweh+BhMXAglriWEtrNKRI rSIAoNX0Y+zNVOoYEVMps2R57XqYCw92 =uREk -----END PGP SIGNATURE----- --T4sUOijqQbZv57TR--