coreutils
[Top][All Lists]
Advanced

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

Re: avoid new GCC-7-exposed warnings


From: Pádraig Brady
Subject: Re: avoid new GCC-7-exposed warnings
Date: Sat, 15 Oct 2016 23:57:26 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0

On 15/10/16 18:11, Pádraig Brady wrote:
> I'll do the follow up patch to use [die] in more places.

A boring copy/replace patch:
84 files changed, 875 insertions(+), 800 deletions(-)
(extra insertions are the inclusion of die.h)

One interesting bit is a new syntax check:

# Usage of error() with an constant to exit() with, should instead use die(),
# as that avoids warnings and may generate better code, due to being apparent
# to the compiler that it doesn't return.
sc_die_EXIT_FAILURE:
       @cd $(srcdir)/src && GIT_PAGER= git grep -E \
           'error \(.*_(FAILURE|INVALID)' \
         && { echo '$(ME): '"Use die() instead of error" 1>&2; \
              exit 1; }  \
         || :

will push later...



reply via email to

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