coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] build: update GCC warnings


From: Jim Meyering
Subject: Re: [PATCH] build: update GCC warnings
Date: Tue, 10 Jun 2014 19:42:52 -0700

On Tue, Jun 10, 2014 at 5:05 PM, Pádraig Brady <address@hidden> wrote:
> * configure.ac: Remove the -Wsuggest-attribute=pure
> enablement on GCC >= 4.7, as that was moot since
> gnulib was already enabling that warning in its default set.
> The false positive was seen with 4.6.2, but confirmed
> not present in 4.6.3, so that's sufficiently old to
> just leave this enabled unconditionally.
>
> Remove the -Wsuggest-attribute={const,noreturn}
> enablement, as gnulib already has those in the default set.
>
> Enable the -Wlogical-op warning for GCC >= 4.8.0
> as that is confirmed OK with coreutils at least, due to
> fixing: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43772
>
> Remove the -Wno-logical-op override since the main
> -Wlogical-op flag is now sufficient to control this warning
> as of GCC 4.6.3 at least.

Nice cleanup.  Thanks.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]