From c2fe618575aaf58ddf36d04d96431d6dc819ef31 Mon Sep 17 00:00:00 2001
From: Tim Peters <tim.peters@gmail.com>
Date: Tue, 30 Oct 2001 23:20:46 +0000
Subject: Fix bad bug in structseq slicing (NULL pointers in result).  Reported
 by Jack Jansen on python-dev. Add simple test case. Move vereq() from
 test_descr to test_support (it's handy!).

---
 Lib/test/test_descr.py     |  6 +-----
 Lib/test/test_structseq.py | 16 ++++++++++++++++
 Lib/test/test_support.py   |  4 ++++
 Objects/structseq.c        |  2 +-
 4 files changed, 22 insertions(+), 6 deletions(-)
 create mode 100644 Lib/test/test_structseq.py

diff --git a/Lib/test/test_descr.py b/Lib/test/test_descr.py
index 4fd11a5..ca09ca9 100644
--- a/Lib/test/test_descr.py
+++ b/Lib/test/test_descr.py
@@ -1,12 +1,8 @@
 # Test enhancements related to descriptors and new-style classes
 
-from test_support import verify, verbose, TestFailed, TESTFN
+from test_support import verify, vereq, verbose, TestFailed, TESTFN
 from copy import deepcopy
 
-def vereq(a, b):
-    if not (a == b):
-        raise TestFailed, "%r == %r" % (a, b)
-
 def veris(a, b):
     if a is not b:
         raise TestFailed, "%r is %r" % (a, b)
diff --git a/Lib/test/test_structseq.py b/Lib/test/test_structseq.py
new file mode 100644
index 0000000..33d3313
--- /dev/null
+++ b/Lib/test/test_structseq.py
@@ -0,0 +1,16 @@
+from test_support import vereq
+
+import time
+
+t = time.gmtime()
+astuple = tuple(t)
+vereq(len(t), len(astuple))
+vereq(t, astuple)
+
+# Check that slicing works the same way; at one point, slicing t[i:j] with
+# 0 < i < j could produce NULLs in the result.
+for i in range(-len(t), len(t)):
+    for j in range(-len(t), len(t)):
+        vereq(t[i:j], astuple[i:j])
+
+XXX more needed
diff --git a/Lib/test/test_support.py b/Lib/test/test_support.py
index bbae1be..83bde3e 100644
--- a/Lib/test/test_support.py
+++ b/Lib/test/test_support.py
@@ -117,6 +117,10 @@ def verify(condition, reason='test failed'):
     if not condition:
         raise TestFailed(reason)
 
+def vereq(a, b):
+    if not (a == b):
+        raise TestFailed, "%r == %r" % (a, b)
+
 def sortdict(dict):
     "Like repr(dict), but in sorted order."
     items = dict.items()
diff --git a/Objects/structseq.c b/Objects/structseq.c
index 50448c2..7ad607b 100644
--- a/Objects/structseq.c
+++ b/Objects/structseq.c
@@ -75,7 +75,7 @@ structseq_slice(PyStructSequence *obj, int low, int high)
 	for(i = low; i < high; ++i) {
 		PyObject *v = obj->ob_item[i];
 		Py_INCREF(v);
-		PyTuple_SET_ITEM(np, i, v);
+		PyTuple_SET_ITEM(np, i-low, v);
 	}
 	return (PyObject *) np;
 }
-- 
cgit v0.12