From owner-freebsd-alpha Fri Nov 30 3:32:52 2001 Delivered-To: freebsd-alpha@freebsd.org Received: from peter3.wemm.org (c1315225-a.plstn1.sfba.home.com [24.14.150.180]) by hub.freebsd.org (Postfix) with ESMTP id 9274737B417; Fri, 30 Nov 2001 03:32:47 -0800 (PST) Received: from overcee.netplex.com.au (overcee.wemm.org [10.0.0.3]) by peter3.wemm.org (8.11.0/8.11.0) with ESMTP id fAUBWlM88941; Fri, 30 Nov 2001 03:32:47 -0800 (PST) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by overcee.netplex.com.au (Postfix) with ESMTP id 53FDC380D; Fri, 30 Nov 2001 03:32:47 -0800 (PST) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Alexey Zelkin Cc: alpha@freebsd.org, dfr@freebsd.org Subject: Re: strtod cleanup (patch) In-Reply-To: <20011127172226.A972@ark.cris.net> Date: Fri, 30 Nov 2001 03:32:47 -0800 From: Peter Wemm Message-Id: <20011130113247.53FDC380D@overcee.netplex.com.au> Sender: owner-freebsd-alpha@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org Alexey Zelkin wrote: > hi, > > On Thu, Nov 22, 2001 at 09:42:41PM -0800, Peter Wemm wrote: > > > > Most of this patch is mechanical merge, but I not sure > > > on types usage correctness (long/int/int32_t) in this chunk: > > > > > > #if BYTE_ORDER == BIG_ENDIAN > > > #define IEEE_BIG_ENDIAN > > > #define Long int32_t > > > #define ULong u_int32_t > > > #else > > > #define IEEE_LITTLE_ENDIAN > > > #define Long long > > > #define ULong unsigned long > > > #endif > > > > This isn't quite right.. The #define Long int32_t etc should be used in > > all cases, including i386. The ia64 and alpha ports are little endian, > > and work best with int32_t. > > > > Also, maybe use: > > typedef int32_t Long; > > typedef u_int32_t ULong; > > .. that way there is no chance of preprocessor side effects with the space > > in ULong. > > Ok. Noted. > > Updated patch included. Can someone who has alpha/ia64 hardware test this > patch ? It should not break anything, but want to be sure. I've been using this on my Alpha. So far it works fine. I used the standard strtod() for a while and discovered it broke my perl build.n (!). If I apply your patches, the perl problem goes away. I am currently having big problems with ia64, I think it is the same thing, but I haven't got a chance to test it out yet. Can you please commit this? It works on Alpha, and I think it solves a nightmare I'm having on ia64. Cheers, -Peter -- Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-alpha" in the body of the message