[Python-checkins] cpython (2.7): Revert "Accept None as start and stop parameters for list.index() and
petri.lehtinen
python-checkins at python.org
Sun Nov 6 20:15:13 CET 2011
http://hg.python.org/cpython/rev/19ffa12ffdd4
changeset: 73407:19ffa12ffdd4
branch: 2.7
user: Petri Lehtinen <petri at digip.org>
date: Sun Nov 06 20:58:50 2011 +0200
summary:
Revert "Accept None as start and stop parameters for list.index() and tuple.index()"
Issue #13340.
files:
Lib/test/list_tests.py | 7 -------
Lib/test/seq_tests.py | 7 -------
Misc/NEWS | 3 ---
Objects/listobject.c | 14 +++-----------
Objects/tupleobject.c | 15 ++++-----------
5 files changed, 7 insertions(+), 39 deletions(-)
diff --git a/Lib/test/list_tests.py b/Lib/test/list_tests.py
--- a/Lib/test/list_tests.py
+++ b/Lib/test/list_tests.py
@@ -365,13 +365,6 @@
self.assertEqual(u.index(0, 3), 3)
self.assertEqual(u.index(0, 3, 4), 3)
self.assertRaises(ValueError, u.index, 2, 0, -10)
- self.assertEqual(u.index(1, None), 4)
- self.assertEqual(u.index(1, None, None), 4)
- self.assertEqual(u.index(1, 0, None), 4)
- self.assertEqual(u.index(1, None, 6), 4)
- self.assertRaises(ValueError, u.index, -1, 3)
- self.assertRaises(ValueError, u.index, -1, 3, None)
- self.assertRaises(ValueError, u.index, 1, None, 4)
self.assertRaises(TypeError, u.index)
diff --git a/Lib/test/seq_tests.py b/Lib/test/seq_tests.py
--- a/Lib/test/seq_tests.py
+++ b/Lib/test/seq_tests.py
@@ -363,13 +363,6 @@
self.assertEqual(u.index(0, 3), 3)
self.assertEqual(u.index(0, 3, 4), 3)
self.assertRaises(ValueError, u.index, 2, 0, -10)
- self.assertEqual(u.index(1, None), 4)
- self.assertEqual(u.index(1, None, None), 4)
- self.assertEqual(u.index(1, 0, None), 4)
- self.assertEqual(u.index(1, None, 6), 4)
- self.assertRaises(ValueError, u.index, -1, 3)
- self.assertRaises(ValueError, u.index, -1, 3, None)
- self.assertRaises(ValueError, u.index, 1, None, 4)
self.assertRaises(TypeError, u.index)
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -11,9 +11,6 @@
- Remove Py3k warning for callable.
-- Issue #13340: Accept None as start and stop parameters for
- list.index() and tuple.index().
-
- Issue #10519: Avoid unnecessary recursive function calls in
setobject.c.
diff --git a/Objects/listobject.c b/Objects/listobject.c
--- a/Objects/listobject.c
+++ b/Objects/listobject.c
@@ -2277,20 +2277,12 @@
{
Py_ssize_t i, start=0, stop=Py_SIZE(self);
PyObject *v, *format_tuple, *err_string;
- PyObject *start_obj = NULL, *stop_obj = NULL;
static PyObject *err_format = NULL;
- if (!PyArg_ParseTuple(args, "O|OO:index", &v, &start_obj, &stop_obj))
+ if (!PyArg_ParseTuple(args, "O|O&O&:index", &v,
+ _PyEval_SliceIndex, &start,
+ _PyEval_SliceIndex, &stop))
return NULL;
-
- if (start_obj != Py_None)
- if (!_PyEval_SliceIndex(start_obj, &start))
- return NULL;
-
- if (stop_obj != Py_None)
- if (!_PyEval_SliceIndex(stop_obj, &stop))
- return NULL;
-
if (start < 0) {
start += Py_SIZE(self);
if (start < 0)
diff --git a/Objects/tupleobject.c b/Objects/tupleobject.c
--- a/Objects/tupleobject.c
+++ b/Objects/tupleobject.c
@@ -510,19 +510,12 @@
tupleindex(PyTupleObject *self, PyObject *args)
{
Py_ssize_t i, start=0, stop=Py_SIZE(self);
- PyObject *v, *start_obj = NULL, *stop_obj = NULL;
+ PyObject *v;
- if (!PyArg_ParseTuple(args, "O|OO:index", &v, &start_obj, &stop_obj))
+ if (!PyArg_ParseTuple(args, "O|O&O&:index", &v,
+ _PyEval_SliceIndex, &start,
+ _PyEval_SliceIndex, &stop))
return NULL;
-
- if (start_obj != Py_None)
- if (!_PyEval_SliceIndex(start_obj, &start))
- return NULL;
-
- if (stop_obj != Py_None)
- if (!_PyEval_SliceIndex(stop_obj, &stop))
- return NULL;
-
if (start < 0) {
start += Py_SIZE(self);
if (start < 0)
--
Repository URL: http://hg.python.org/cpython
More information about the Python-checkins
mailing list