Date: Sun, 7 Oct 2001 13:25:11 +0300 From: Alexey Zelkin <phantom@FreeBSD.org> To: Doug Rabson <dfr@nlsystems.com> Cc: Jake Burkholder <jake@locore.ca>, Doug Rabson <dfr@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libc/stdlib strtod.c Message-ID: <20011007132511.A32275@ark.cris.net> In-Reply-To: <20011006173708.B530-100000@salmon.nlsystems.com>; from dfr@nlsystems.com on Sat, Oct 06, 2001 at 05:37:53PM %2B0100 References: <20011006121653.D7732@locore.ca> <20011006173708.B530-100000@salmon.nlsystems.com>
next in thread | previous in thread | raw e-mail | index | archive | help
hi,
On Sat, Oct 06, 2001 at 05:37:53PM +0100, Doug Rabson wrote:
> On Sat, 6 Oct 2001, Jake Burkholder wrote:
>
> > Apparently, On Sat, Oct 06, 2001 at 08:58:54AM -0700,
> > Doug Rabson said words to the effect of;
> >
> > > dfr 2001/10/06 08:58:54 PDT
> > >
> > > Modified files:
> > > lib/libc/stdlib strtod.c
> > > Log:
> > > Make this work on ia64. I have no idea why it works on alpha - it
> > > shouldn't.
> > >
> > > Revision Changes Path
> > > 1.9 +8 -2 src/lib/libc/stdlib/strtod.c
> >
> > I assume that's why we have netbsd_strtod.c and a not so temporary
> > hack in the stdlib Makefile.inc.
> >
> > .if ${MACHINE_ARCH} == "alpha"
> > # XXX Temporary until the assumption that a long is 32-bits is resolved
> > # XXX FreeBSD's code. NetBSD kludged this with Long = int32_t and
> > # XXX ULong = u_int32_t
> > SRCS+= netbsd_strtod.c
> > .else
> > SRCS+= strtod.c
> > .endif
>
> Oh yes, I remember that. Hehe. Perhaps we should always use
> netbsd_strtod.c (or maybe get a fresh copy from NetBSD-current).
Few weeks ago I made a work on review and merge functional parts of
netbsd's strtod(3) (both used in out tree, as netbsd_strtod.c, and
recent NetBSD) and our strtod(3). My patch eliminates need of having
netbsd's version of strtod in our tree. I'll find this patch and post to you
for review and testing because it's mainly affecting non i386 architectures.
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20011007132511.A32275>