]> granicus.if.org Git - python/commitdiff
Revert "Accept None as start and stop parameters for list.index() and tuple.index()"
authorPetri Lehtinen <petri@digip.org>
Sun, 6 Nov 2011 19:05:41 +0000 (21:05 +0200)
committerPetri Lehtinen <petri@digip.org>
Sun, 6 Nov 2011 19:06:10 +0000 (21:06 +0200)
Issue #13340.

1  2 
Lib/test/list_tests.py
Misc/NEWS
Objects/listobject.c
Objects/tupleobject.c

Simple merge
diff --cc Misc/NEWS
index 82edaa4f783c81d1caf7d91da8558cd4c5a6adbd,2923982755a863ec4ed38e226553ca7a20f6ab8a..7346348867920fc02a4f2795537f6a9c604c0ede
+++ b/Misc/NEWS
@@@ -16,11 -13,6 +16,8 @@@ Core and Builtin
  - Issue #13342: input() used to ignore sys.stdin's and sys.stdout's unicode
    error handler in interactive mode (when calling into PyOS_Readline()).
  
- - Issue #13340: Accept None as start and stop parameters for
-   list.index() and tuple.index().
 +- Issue #9896: Add start, stop, and step attributes to range objects.
 +
  - Issue #13343: Fix a SystemError when a lambda expression uses a global
    variable in the default value of a keyword-only argument:
    (lambda *, arg=GLOBAL_NAME: None)
index cf4f2766d579462dec22e4f59421cf896bf19761,00de597e5686221b25563cbf51d05be87a9bb22f..69a632d5289d485385a36dd2b7c5adf5ee366754
@@@ -2121,20 -2108,13 +2121,12 @@@ static PyObject 
  listindex(PyListObject *self, PyObject *args)
  {
      Py_ssize_t i, start=0, stop=Py_SIZE(self);
 -    PyObject *v, *format_tuple, *err_string;
 -    static PyObject *err_format = NULL;
 +    PyObject *v;
-     PyObject *start_obj = NULL, *stop_obj = 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)
Simple merge