Re: proposed new module intprops-test

2011-05-20 Thread Paul Eggert
> With just the first 2 out of 3 commits (partial revert and work around IRIX > 6.5 > cc bug), IRIX 6.5 cc would still have complained ("The indicated constant > value > is not known.") about a combination of signed and unsigned integers, this time > with '|', not '+' ... and HP-UX cc would still

Re: proposed new module intprops-test

2011-05-20 Thread Bruno Haible
Hi Paul, > Here's what I committed: With these patches, the test compiles and passes now also on HP-UX 11.23 with cc and IRIX 6.5 with cc. With just the first 2 out of 3 commits (partial revert and work around IRIX 6.5 cc bug), IRIX 6.5 cc would still have complained ("The indicated constant val

Re: proposed new module intprops-test

2011-05-19 Thread Paul Eggert
On 05/19/11 06:07, Bruno Haible wrote: > Thanks, this patch fixed the errors on OSF/1 5.1 and reduced the errors on > Solaris 9 to only one:... Yes, thanks, I've committed something which I hope fixes that; see below. > This part of the patch has no effect on HP-UX, IRIX, and neither on > OSF/1 a

Re: proposed new module intprops-test

2011-05-19 Thread Bruno Haible
Hi Paul, > 2011-05-19 Paul Eggert > > + intprops: work around C compiler bugs > + * lib/intprops.h (INT_MULTIPLY_RANGE_OVERFLOW): Work around compiler > + bug in Sun C 5.11 2010/08/13 and other compilers; see > +

Re: proposed new module intprops-test

2011-05-19 Thread Paul Eggert
Thanks for testing that. I replicated the problem on Solaris 10 with Sun C 5.11 and Solaris 8 with Sun C 5.8. The following patches fixed it for me. I can't easily test HP-UX and IRIX but I tried to fix all the problems I could deduce remotely. Apparently these compilers mishandle constants lik

Re: proposed new module intprops-test

2011-05-18 Thread Bruno Haible
Hi Paul, > > ./gnulib-tool --with-tests --test intprops > > Thanks, I ran that and it worked. It fails to compile on HP-UX 11.23 and 11.31 with cc, on IRIX 6.5 with cc, on OSF/1 5.1 with cc, and on Solaris 9 and 10 (SPARC) with cc. Here are the error messages on HP-UX 11.23 with cc: cc

Re: proposed new module intprops-test

2011-05-10 Thread Paul Eggert
On 05/10/11 10:07, Eric Blake wrote: > ./gnulib-tool --with-tests --test intprops Thanks, I ran that and it worked. > verify (! TYPE_IS_INTEGER (void *)); No, TYPE_IS_INTEGER is defined only for arithmetic types, so I left that alone. > verify (TYPE_SIGNED (double)); Thanks, I added that (see

Re: proposed new module intprops-test

2011-05-10 Thread Eric Blake
On 05/10/2011 10:57 AM, Paul Eggert wrote: > Here's a proposed module to test 'intprops'. It assumes the revised > patch for integer overflow checking that I emailed a few minutes ago. > > I don't know the recommended way to test this with gnulib-tool; > I read the --help output and the documenta