[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: enable -Werror for lib/ in coreutils
From: |
Jim Meyering |
Subject: |
Re: enable -Werror for lib/ in coreutils |
Date: |
Tue, 27 Oct 2009 14:22:16 +0100 |
Eric Blake wrote:
> According to Jim Meyering on 10/27/2009 6:24 AM:
>> +
>> + AC_SUBST([GNULIB_WARN_CFLAGS])
>> + GNULIB_WARN_CFLAGS="$WARN_CFLAGS -Wno-missing-prototypes
>> -Wno-uninitialized -Wno-unused-macros -Wno-old-style-definition"
>
> Shouldn't this use gl_MANYWARN_COMPLEMENT([var], [$ws],
> [-Wmissing-prototypes -Wuninitialized...]), rather than assuming that the
> compiler will understand -Wno-missing-prototypes and friends?
Thanks!
Yes, that will be better indeed. Will adjust.
>> ++++ b/lib/isnan.c
>> +@@ -67,6 +67,8 @@
>> + ((sizeof (DOUBLE) + sizeof (unsigned int) - 1) / sizeof (unsigned int))
>> + typedef union { DOUBLE value; unsigned int word[NWORDS]; } memory_double;
>> +
>> ++int FUNC (DOUBLE x);
>
> Why are there no header files declaring rpl_isnan? Isn't a better fix to
> patch math.in.h to declare everything that we will be providing?
I thought I'd reported it already, but didn't go back and check.
It seems I did not, after all.
- enable -Werror for lib/ in coreutils, Jim Meyering, 2009/10/27
- Re: enable -Werror for lib/ in coreutils, Eric Blake, 2009/10/27
- Re: enable -Werror for lib/ in coreutils, Pádraig Brady, 2009/10/27
- Re: enable -Werror for lib/ in coreutils, Bruno Haible, 2009/10/27
- Re: enable -Werror for lib/ in coreutils, Jim Meyering, 2009/10/28
- Re: enable -Werror for lib/ in coreutils, Paolo Bonzini, 2009/10/28