[Python-checkins] r69441 - in python/branches/release30-maint: Objects/object.c
mark.dickinson
python-checkins at python.org
Sun Feb 8 16:11:29 CET 2009
Author: mark.dickinson
Date: Sun Feb 8 16:11:29 2009
New Revision: 69441
Log:
Merged revisions 69440 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k
................
r69440 | mark.dickinson | 2009-02-08 15:09:21 +0000 (Sun, 08 Feb 2009) | 17 lines
Merged revisions 69436 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r69436 | mark.dickinson | 2009-02-08 14:42:28 +0000 (Sun, 08 Feb 2009) | 10 lines
Issue #789290: make sure that hash(2**63) == hash(2.**63) on 64-bit
platforms. The previous code was fragile, depending on the twin
accidents that:
(1) in C, casting the double value 2.**63 to long returns the integer
value -2**63, and
(2) in Python, hash(-2**63) == hash(2**63).
There's already a test for this in test_hash.
........
................
Modified:
python/branches/release30-maint/ (props changed)
python/branches/release30-maint/Objects/object.c
Modified: python/branches/release30-maint/Objects/object.c
==============================================================================
--- python/branches/release30-maint/Objects/object.c (original)
+++ python/branches/release30-maint/Objects/object.c Sun Feb 8 16:11:29 2009
@@ -650,7 +650,7 @@
fractpart = modf(v, &intpart);
if (fractpart == 0.0) {
/* This must return the same hash as an equal int or long. */
- if (intpart > LONG_MAX || -intpart > LONG_MAX) {
+ if (intpart > LONG_MAX/2 || -intpart > LONG_MAX/2) {
/* Convert to long and use its hash. */
PyObject *plong; /* converted to Python long */
if (Py_IS_INFINITY(intpart))
More information about the Python-checkins
mailing list