X-Authentication-Warning:	delorie.com: mail set sender to djgpp-workers-bounces using -f
Sender:	rich AT phekda DOT freeserve DOT co DOT uk
Message-ID:	<3FAFF6E3.B2A1DF62@phekda.freeserve.co.uk>
Date:	Mon, 10 Nov 2003 20:36:51 +0000
From:	Richard Dawe <rich AT phekda DOT freeserve DOT co DOT uk>
X-Mailer:	Mozilla 4.77 [en] (X11; U; Linux 2.2.23 i586)
X-Accept-Language:	de,fr
MIME-Version:	1.0
To:	djgpp-workers AT delorie DOT com
Subject:	Re: Fix some warnings in Cygnus test vector generation [PATCH]
References:	<d DOT 1bb62c77 DOT 2ce0039d AT aol DOT com>
Reply-To:	djgpp-workers AT delorie DOT com

Hello.

In a message dated 11/9/2003 3:13:54 PM Eastern Standard Time,
rich AT phekda DOT freeserve DOT co DOT uk writes:
> > The header genmathv.h doesn't include enough braces in the definitions
> > it generates for test vectors. Below is a patch to fix this.
> > 
> > Even after applying this patch there are still warnings, because
> > the entries terminating the lists don't have enough braces.
> I guess fixing "0" to be "{0}" is enough, but I didn't feel like
> > fixing tens of files. I'll do that some other time.
> > 
> > OK to commit?
> 
> This will be repaired in the new version of the test generators that I'm
> working on.  None of the old code for the float and double functions is
> used when the C code is generated.
>
> I have a copy of the old version of genmathv.h that I used when Eli
> discovered how to fix the problem.  It is attached. I hope it helps.  It
> generated clean C code from the tgen subdirectory.  Problem is, it also
> contains some new code for the long double functions that might get in
> the way.  I believe the new code is blocked out by #ifdef statements.
[snip]

Thanks. It looks like I've fixed it in the same way. If there are no further
comments, I will commit my patch at the weekend.

Thanks, bye, Rich =]

-- 
Richard Dawe [ http://www.phekda.freeserve.co.uk/richdawe/ ]

"You can't evaluate a man by logic alone." -- McCoy, "I, Mudd", Star Trek

- Raw text -