GNU bug report logs - #24300
intprops.h GL_INT_OP_WRAPV_LONGISH one too many ")"

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: "John E. Malmberg" <wb8tyw <at> qsl.net>
Subject: bug#24300: closed (Re: [bug-diffutils] bug#24300: intprops.h
 GL_INT_OP_WRAPV_LONGISH one too many ")")
Date: Thu, 25 Aug 2016 03:54:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#24300: intprops.h GL_INT_OP_WRAPV_LONGISH one too many ")"

which was filed against the diffutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 24300 <at> debbugs.gnu.org.

-- 
24300: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=24300
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: "John E. Malmberg" <wb8tyw <at> qsl.net>, 24300-done <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24300: intprops.h GL_INT_OP_WRAPV_LONGISH one
 too many ")"
Date: Wed, 24 Aug 2016 20:53:28 -0700

[Message part 3 (text/plain, inline)]

John E. Malmberg wrote:
> I am currently working around this issue by adding:
>
> #define LLONG_MAX __INT64_MAX
> #define LLONG_MIN __INT64_MIN

Thanks, I installed into Gnulib the attached patch, which should be equivalent.

[0001-intprops-port-to-OpenVMS.patch (text/x-diff, attachment)]

[Message part 5 (message/rfc822, inline)]

From: "John E. Malmberg" <wb8tyw <at> qsl.net>
To: bug-diffutils <at> gnu.org
Subject: intprops.h GL_INT_OP_WRAPV_LONGISH one too many ")"
Date: Wed, 24 Aug 2016 08:30:24 -0500

Greetings,

While building diffutils 3.5 on OpenVMS/IA64 8.4, I discovered:

 I1        5525 #  define _GL_INT_OP_WRAPV_LONGISH(a, b, r, op, overflow) \
 I1        5526     _GL_INT_OP_CALC (a, b, r, op, overflow, unsigned 
long int, \
 I1        5527                      long int, LONG_MIN, LONG_MAX))


This was tripped because the VMS limits.h file does not define LLONG_MIN 
and LLONG_MAX.


I am currently working around this issue by adding:

#define LLONG_MAX __INT64_MAX
#define LLONG_MIN __INT64_MIN

To a pre-include header file on the modules, which works for the OpenVMS 
Alpha and IA64 builds.

Regards,
-John

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.