Mark Dickinson added the comment:

The test portion of the existing patch doesn't apply cleanly to 2.7 tip. Here's 
an updated patch that does, with slightly expanded tests and a Misc/NEWS entry.

There was discussion above about two overflow cases: C long to C int, and C int 
to byte, but with this patch there's now a 3rd overflow case to consider, 
namely Python long to C long.  In this case PyInt_AsLong sets an exception and 
returns -1, and since we don't check for exceptions that -1 then gets used as 
the exit code. I'm not sure whether that's the right thing to do or not, but it 
does match what Python 3 currently does, and that's good enough for me.

----------
Added file: http://bugs.python.org/file46481/exit2.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue14376>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to