Re: test-intprops.c: many new warnings

2011-05-25 Thread Paul Eggert
On 05/24/11 17:06, Eric Blake wrote: > Would it be possible instead to write forwarding macros? Sorry, I don't see how. The macros might expand to runtime tests. > Otherwise, we are just disabling the warnings for this test, but the > warnings will still byte us in regular code. -Wtype-limits i

Re: test-intprops.c: many new warnings

2011-05-24 Thread Jim Meyering
Paul Eggert wrote: > On 05/24/11 12:36, Jim Meyering wrote: >> "make check" was inundated with new warnings. >> Nearly 500 lines worth. > > Thanks, I fixed those by pushing the following two patches: Those look fine and work for me. Thanks!

Re: test-intprops.c: many new warnings

2011-05-24 Thread Eric Blake
On 05/24/2011 05:50 PM, Paul Eggert wrote: > +++ b/tests/test-intprops.c > @@ -16,6 +16,12 @@ > > /* Written by Paul Eggert. */ > > +/* Tell gcc not to warn about the many (X < 0) expressions that > + the overflow macros expand to. */ > +#if (__GNUC__ == 4 && 3 <= __GNUC_MINOR__) || 4 < __

Re: test-intprops.c: many new warnings

2011-05-24 Thread Paul Eggert
On 05/24/11 12:36, Jim Meyering wrote: > "make check" was inundated with new warnings. > Nearly 500 lines worth. Thanks, I fixed those by pushing the following two patches: --- ChangeLog |9 + lib/intprops.h |8 2 files changed, 13 insertions(+), 4 deletions(-) di

test-intprops.c: many new warnings

2011-05-24 Thread Jim Meyering
Still trying the latest from gnulib via coreutils, "make check" was inundated with new warnings. Nearly 500 lines worth. Of course, I could simply turn off the warnings and/or -Werror when building in gnulib-tests/, but I'd rather not. I'll defer "upgrading to the latest" for now ;-) ... Making c