Only call sq_length in Sequence_GetItem for negative index.

08ef9d98 · Guido van Rossum · 115c1144 · 08ef9d98
Commit 08ef9d98 authored Nov 09, 1996 by Guido van Rossum
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 4 deletions

Objects/abstract.c Objects/abstract.c +6 -4

No files found.
--- a/Objects/abstract.c
+++ b/Objects/abstract.c
@@ -666,12 +666,14 @@ PySequence_GetItem(s, i)
  if(! s) return Py_ReturnNullError();
-  if(! ((m=s->ob_type->tp_as_sequence) && m->sq_length && m->sq_item))
+  if(! ((m=s->ob_type->tp_as_sequence) && m->sq_item))
    return Py_ReturnMethodError("__getitem__");  
+  if(i < 0)
+    {
      if(0 > (l=m->sq_length(s))) return NULL;
+      i += l;
-  if(i < 0) i += l;
+    }
  return m->sq_item(s,i);
 }