Kaydet (Commit) c10c9d0d authored tarafından Tim Peters's avatar Tim Peters

"Conceptual" merge of rev 51711 from the 2.5 branch.

i_divmod():  As discussed on Python-Dev, changed the overflow
checking to live happily with recent gcc optimizations that
assume signed integer arithmetic never overflows.

This differs from the corresponding change on the 2.5 and 2.4
branches, using a less obscure approach, but one that /may/
tickle platform idiocies in their definitions of LONG_MIN.
The 2.4 + 2.5 change avoided introducing a dependence on
LONG_MIN, at the cost of substantially goofier code.
üst a0c95fa4
...@@ -12,6 +12,9 @@ What's New in Python 2.6 alpha 1? ...@@ -12,6 +12,9 @@ What's New in Python 2.6 alpha 1?
Core and builtins Core and builtins
----------------- -----------------
- Overflow checking code in integer division ran afoul of new gcc
optimizations. Changed to be more standard-conforming.
- Patch #1542451: disallow continue anywhere under a finally. - Patch #1542451: disallow continue anywhere under a finally.
...@@ -136,7 +139,7 @@ Library ...@@ -136,7 +139,7 @@ Library
- The __repr__ method of a NULL ctypes.py_object() no longer raises - The __repr__ method of a NULL ctypes.py_object() no longer raises
an exception. an exception.
- uuid.UUID now has a bytes_le attribute. This returns the UUID in - uuid.UUID now has a bytes_le attribute. This returns the UUID in
little-endian byte order for Windows. In addition, uuid.py gained some little-endian byte order for Windows. In addition, uuid.py gained some
workarounds for clocks with low resolution, to stop the code yielding workarounds for clocks with low resolution, to stop the code yielding
duplicate UUIDs. duplicate UUIDs.
...@@ -295,7 +298,7 @@ Library ...@@ -295,7 +298,7 @@ Library
- Bug #1002398: The documentation for os.path.sameopenfile now correctly - Bug #1002398: The documentation for os.path.sameopenfile now correctly
refers to file descriptors, not file objects. refers to file descriptors, not file objects.
- The renaming of the xml package to xmlcore, and the import hackery done - The renaming of the xml package to xmlcore, and the import hackery done
to make it appear at both names, has been removed. Bug #1511497, to make it appear at both names, has been removed. Bug #1511497,
#1513611, and probably others. #1513611, and probably others.
......
...@@ -565,7 +565,7 @@ i_divmod(register long x, register long y, ...@@ -565,7 +565,7 @@ i_divmod(register long x, register long y,
return DIVMOD_ERROR; return DIVMOD_ERROR;
} }
/* (-sys.maxint-1)/-1 is the only overflow case. */ /* (-sys.maxint-1)/-1 is the only overflow case. */
if (y == -1 && x < 0 && x == -x) if (y == -1 && x == LONG_MIN)
return DIVMOD_OVERFLOW; return DIVMOD_OVERFLOW;
xdivy = x / y; xdivy = x / y;
xmody = x - xdivy * y; xmody = x - xdivy * y;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment