Re: [PATCH] xalloc: avoid GCC 5.1 warning on 32 bit

2015-10-03 Thread Pádraig Brady
On 03/10/15 08:39, Paul Eggert wrote: > Pádraig Brady wrote: >> xalloc is >> included into the coreutils compilation units and so >> needs this guard. > > That's a problem in coreutils, not in gnulib. Right. I missed the fact it applied only to a single use of xnrealloc. > Please don't give much

Re: [PATCH] xalloc: avoid GCC 5.1 warning on 32 bit

2015-10-03 Thread Paul Eggert
Pádraig Brady wrote: xalloc is included into the coreutils compilation units and so needs this guard. That's a problem in coreutils, not in gnulib. Please don't give much weight to those particular compiler warnings. They're often false alarms, as these were. There is nothing wrong when a

Re: [PATCH] xalloc: avoid GCC 5.1 warning on 32 bit

2015-10-02 Thread Pádraig Brady
On 02/10/15 21:22, Paul Eggert wrote: > On 10/02/2015 12:24 PM, Pádraig Brady wrote: >> +/* GCC 5.1 gives an erroneous warning on 32 bit for xalloc_oversized(): >> + "assuming signed overflow does not occur when simplifying conditional". >> */ >> +#if (__GNUC__ == 4 && 6 <= __GNUC_MINOR__) || 4

Re: [PATCH] xalloc: avoid GCC 5.1 warning on 32 bit

2015-10-02 Thread Paul Eggert
On 10/02/2015 12:24 PM, Pádraig Brady wrote: +/* GCC 5.1 gives an erroneous warning on 32 bit for xalloc_oversized(): + "assuming signed overflow does not occur when simplifying conditional". */ +#if (__GNUC__ == 4 && 6 <= __GNUC_MINOR__) || 4 < __GNUC__ +# pragma GCC diagnostic push +# pragma

[PATCH] xalloc: avoid GCC 5.1 warning on 32 bit

2015-10-02 Thread Pádraig Brady
* lib/xalloc.h: Disable -Wstrict-overflow for uses of xalloc_oversized(), which was seen to give this warning on GCC 5.1 on 32 bit: "assuming signed overflow does not occur when simplifying conditional". --- ChangeLog| 8 lib/xalloc.h | 10 ++ 2 files changed, 18 insertions(+