From: Kristján Valur Jónsson <sweskman@gmail.com>
Date: Sat, 12 Sep 2015 15:30:23 +0000 (+0000)
Subject: Issue #25021: Merge from 3.3 to 3.4
X-Git-Tag: v3.5.1rc1~368^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=95c3e6cb2243acc0a102d9fa59efece1427a3936;p=python

Issue #25021: Merge from 3.3 to 3.4
---

95c3e6cb2243acc0a102d9fa59efece1427a3936
diff --cc Lib/test/test_itertools.py
index 3aed77988e,35391c96b4..993438c9ce
--- a/Lib/test/test_itertools.py
+++ b/Lib/test/test_itertools.py
@@@ -982,9 -957,18 +982,19 @@@ class TestBasicOps(unittest.TestCase)
              ]:
              self.assertEqual(list(copy.copy(product(*args))), result)
              self.assertEqual(list(copy.deepcopy(product(*args))), result)
 -            self.pickletest(product(*args))
 +            for proto in range(pickle.HIGHEST_PROTOCOL + 1):
 +                self.pickletest(proto, product(*args))
  
+     def test_product_issue_25021(self):
+         # test that indices are properly clamped to the length of the tuples
+         p = product((1, 2),(3,))
+         p.__setstate__((0, 0x1000))  # will access tuple element 1 if not clamped
+         self.assertEqual(next(p), (2, 3))
+         # test that empty tuple in the list will result in an immediate StopIteration
+         p = product((1, 2), (), (3,))
+         p.__setstate__((0, 0, 0x1000))  # will access tuple element 1 if not clamped
+         self.assertRaises(StopIteration, next, p)
+ 
      def test_repeat(self):
          self.assertEqual(list(repeat(object='a', times=3)), ['a', 'a', 'a'])
          self.assertEqual(lzip(range(3),repeat('a')),