Index: Objects/abstract.c =================================================================== --- Objects/abstract.c (revision 68545) +++ Objects/abstract.c (working copy) @@ -1379,19 +1379,7 @@ } return res; } - if (m && m->nb_long) { /* This should include subclasses of long */ - /* Classic classes always take this branch. */ - PyObject *res = m->nb_long(o); - if (res && !PyLong_Check(res)) { - PyErr_Format(PyExc_TypeError, - "__long__ returned non-long (type %.200s)", - res->ob_type->tp_name); - Py_DECREF(res); - return NULL; - } - return res; - } - if (PyLong_Check(o)) /* A long subclass without nb_long */ + if (PyLong_Check(o)) /* An int subclass without nb_int */ return _PyLong_Copy((PyLongObject *)o); trunc_func = PyObject_GetAttr(o, trunc_name); if (trunc_func) { Index: Misc/NEWS =================================================================== --- Misc/NEWS (revision 68545) +++ Misc/NEWS (working copy) @@ -12,6 +12,10 @@ Core and Builtins ----------------- +- Issue #4910: Builtin int() function and PyNumber_Long/PyNumber_Int API + function no longer attempt to call the __long__ slot to convert an object + to an integer. Only the __int__ and __trunc__ slots are examined. + - Issue #4893: Use NT threading on CE. - Issue #4915: Port sysmodule to Windows CE. Index: Lib/test/test_long.py =================================================================== --- Lib/test/test_long.py (revision 68545) +++ Lib/test/test_long.py (working copy) @@ -367,7 +367,7 @@ def test_conversion(self): - # Test __long__() + # Test __int__() class ClassicMissingMethods: pass self.assertRaises(TypeError, int, ClassicMissingMethods()) @@ -410,18 +410,32 @@ class Classic: pass for base in (object, Classic): - class LongOverridesTrunc(base): - def __long__(self): + class IntOverridesTrunc(base): + def __int__(self): return 42 def __trunc__(self): return -12 - self.assertEqual(int(LongOverridesTrunc()), 42) + self.assertEqual(int(IntOverridesTrunc()), 42) class JustTrunc(base): def __trunc__(self): return 42 self.assertEqual(int(JustTrunc()), 42) + class JustLong(base): + # test that __long__ no longer used in 3.x + def __long__(self): + return 42 + self.assertRaises(TypeError, int, JustLong()) + + class LongTrunc(base): + # __long__ should be ignored in 3.x + def __long__(self): + return 42 + def __trunc__(self): + return 1729 + self.assertEqual(int(LongTrunc()), 1729) + for trunc_result_base in (object, Classic): class Integral(trunc_result_base): def __int__(self):