[ python-Bugs-1502750 ] PyArg_ParseTuple(args, "i") and sys.maxint

[SourceForge.net]

Bugs item #1502750, was opened at 2006-06-08 10:42
Message generated for change (Comment added) made by lemburg
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1502750&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Python Library
Group: Python 2.5
Status: Open
Resolution: None
Priority: 5
Submitted By: M.-A. Lemburg (lemburg)
Assigned to: Georg Brandl (gbrandl)
Summary: PyArg_ParseTuple(args, "i") and sys.maxint

Initial Comment:
The argument parser seems to have trouble with
sys.maxint on 64-bit platforms:

>>> import sys
>>> sys.maxint
9223372036854775807
>>> sys.setcheckinterval(sys.maxint)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
OverflowError: signed integer is greater than maximum

>>> sys.setcheckinterval(2**31-1)
>>> sys.setcheckinterval(2**31)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
OverflowError: signed integer is greater than maximum

Looks like INT_MAX isn't the right value to test against.

This is on AMD64, Linux2.6, gcc 3.3.


----------------------------------------------------------------------

>Comment By: M.-A. Lemburg (lemburg)
Date: 2006-06-08 16:00

Message:
Logged In: YES 
user_id=38388

Yes. Sorry for the confusion.

ints are 32 bit and longs 64 bit on 64bit Linux (and other
64bit Unix systems).

Since Python uses C longs for storing integers, sys.maxint
correctly gives LONG_MAX.


----------------------------------------------------------------------

Comment By: Georg Brandl (gbrandl)
Date: 2006-06-08 15:52

Message:
Logged In: YES 
user_id=849994

To be sure: ints are 32-bit and longs 64-bit?


----------------------------------------------------------------------

Comment By: M.-A. Lemburg (lemburg)
Date: 2006-06-08 15:45

Message:
Logged In: YES 
user_id=38388

Strike that last comment about ints haveing 64-bit. I got
carried away by Python using C longs for ints.


----------------------------------------------------------------------

Comment By: M.-A. Lemburg (lemburg)
Date: 2006-06-08 15:39

Message:
Logged In: YES 
user_id=38388

Reopened: the bug is still there.

Note that a C integer does have 64-bits on a 64-bit Linux
system :-) (unlike on a Win64 system).


----------------------------------------------------------------------

Comment By: Georg Brandl (gbrandl)
Date: 2006-06-08 15:32

Message:
Logged In: YES 
user_id=849994

The bug isn't in getargs. Format code "i" wants a C integer.

Question is whether sys.setcheckinterval and perhaps dozens
of other functions should be changed to accept "l".

Changing to Invalid. Reverted checkin.

----------------------------------------------------------------------

Comment By: Georg Brandl (gbrandl)
Date: 2006-06-08 14:47

Message:
Logged In: YES 
user_id=849994

No, INT_MAX is definitely not the right value. Changed to
LONG_MAX and LONG_MIN in rev. 46741, 46742 (2.4).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1502750&group_id=5470