mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-07-29 06:05:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Don't allow keyword arguments to reversed().

Browse source
This commit is contained in:
Georg Brandl 2008-05-16 13:24:29 +00:00
parent 88107dafef
commit ecf9091f59
2 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Lib/test/test_enumerate.py
View file

@ -138,6 +138,8 @@ class TestReversed(unittest.TestCase):
for data in 'abc', range(5), tuple(enumerate('abc')), A(), xrange(1,17,5): for data in 'abc', range(5), tuple(enumerate('abc')), A(), xrange(1,17,5):
self.assertEqual(list(data)[::-1], list(reversed(data))) self.assertEqual(list(data)[::-1], list(reversed(data)))
self.assertRaises(TypeError, reversed, {}) self.assertRaises(TypeError, reversed, {})
# don't allow keyword arguments
self.assertRaises(TypeError, reversed, [], a=1)
def test_xrange_optimization(self): def test_xrange_optimization(self):
x = xrange(1) x = xrange(1)

5
Objects/enumobject.c
View file

@ -224,7 +224,10 @@ reversed_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
PyObject *seq; PyObject *seq;
reversedobject *ro; reversedobject *ro;
if (!PyArg_UnpackTuple(args, "reversed", 1, 1, &seq)) if (type == &PyReversed_Type && !_PyArg_NoKeywords("reversed()", kwds))
return NULL;
if (!PyArg_UnpackTuple(args, "reversed", 1, 1, &seq) )
return NULL; return NULL;
if (PyObject_HasAttrString(seq, "__reversed__")) if (PyObject_HasAttrString(seq, "__reversed__"))