diff --git a/Lib/test/test_builtin.py b/Lib/test/test_builtin.py index d2d4fa9fcf1..4d9b5c320a0 100644 --- a/Lib/test/test_builtin.py +++ b/Lib/test/test_builtin.py @@ -1033,6 +1033,8 @@ def __index__(self): self.assertEqual(range(3).count(1), 1) self.assertEqual(range(3).count(2), 1) self.assertEqual(range(3).count(3), 0) + self.assertIs(type(range(3).count(-1)), int) + self.assertIs(type(range(3).count(1)), int) self.assertEqual(range(10**20).count(1), 1) self.assertEqual(range(10**20).count(10**20), 0) diff --git a/Misc/NEWS b/Misc/NEWS index f27c4b0d3f6..04d624c68d2 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -10,6 +10,8 @@ What's New in Python 3.2 Beta 1? Core and Builtins ----------------- +- Issue #10474: range().count() should return integers. + - Issue #10255: Fix reference leak in Py_InitializeEx(). Patch by Neil Schemenauer. diff --git a/Objects/rangeobject.c b/Objects/rangeobject.c index 965038635c3..2188dfba597 100644 --- a/Objects/rangeobject.c +++ b/Objects/rangeobject.c @@ -338,9 +338,9 @@ range_count(rangeobject *r, PyObject *ob) { if (PyLong_CheckExact(ob) || PyBool_Check(ob)) { if (range_contains_long(r, ob)) - Py_RETURN_TRUE; + return PyLong_FromLong(1); else - Py_RETURN_FALSE; + return PyLong_FromLong(0); } else { Py_ssize_t count; count = _PySequence_IterSearch((PyObject*)r, ob, PY_ITERSEARCH_COUNT);