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 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: Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit 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