[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: SA_RESETHAND
From: |
Bruno Haible |
Subject: |
Re: SA_RESETHAND |
Date: |
Thu, 21 May 2020 22:59:08 +0200 |
User-agent: |
KMail/5.1.3 (Linux/4.4.0-177-generic; KDE/5.18.0; x86_64; ; ) |
Paul Eggert wrote:
> > - Should glibc define SA_RESETHAND as ((int)0x80000000) ?
> > Then SA_RESETHAND could not be used in preprocessor directives any more.
>
> POSIX would allow that, as it doesn't require SA_RESETHAND to be usable
> in preprocessor directives. However, too much software uses it that way
> anyway (e.g., squid/src/tools.cc has "#if SA_RESETHAND == 0 &&
> !_SQUID_WINDOWS_"). So I have my doubts whether this change would be
> adopted.
>
> > - Should clang be silent about this case of implicit conversion?
>
> That would solve the problem, although the people who want lots of
> warnings might want one here too.
>
> > - Should we discourage users from using
> > -fsanitize=implicit-integer-sign-change?
>
> For me that flag tends to cause more problem than it cures. So we could
> tell people that Gnulib won't worry about that warning.
Thanks for your evaluation. I think this should best be fixed by clang, and
therefore have registered a clang bug:
https://bugs.llvm.org/show_bug.cgi?id=46025
Bruno
- Re: Fix calloc.m4 test, (continued)
- Fix invalid use of __builtin_isnanf and __builtin_isnanl, Bruno Haible, 2020/05/23
- Fix calloc-gnu configure results, Bruno Haible, 2020/05/23
- Re: Fix memleak in getdelim.m4, Bruno Haible, 2020/05/23
- Re: Fix memleak in getdelim.m4, Tim Rühsen, 2020/05/24
- SA_RESETHAND, Bruno Haible, 2020/05/21
- Re: SA_RESETHAND, Paul Eggert, 2020/05/21
- Re: SA_RESETHAND,
Bruno Haible <=
- Re: Fix sanitizer error in fchownat.m4, Bruno Haible, 2020/05/21
- Fix memleak in glob.m4, Bruno Haible, 2020/05/21
- Fix memleak in regex.m4, Bruno Haible, 2020/05/21
- Re: Fix memleak in getdelim.m4, Bruno Haible, 2020/05/21
- Re: Fix memleak in getdelim.m4, Paul Eggert, 2020/05/21
- Re: Fix memleak in getdelim.m4, Bruno Haible, 2020/05/21
Re: Fix memleak in getdelim.m4, Bruno Haible, 2020/05/21