| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -967,6 +967,12 @@ def test_repeat(self): | |||
| 967 | 967 | self.assertEqual(take(2, copy.deepcopy(c)), list('a' * 2)) | |
| 968 | 968 | self.pickletest(repeat(object='a', times=10)) | |
| 969 | 969 | ||
| 970 | + def test_repeat_with_negative_times(self): | ||
| 971 | + self.assertEqual(repr(repeat('a', -1)), "repeat('a', 0)") | ||
| 972 | + self.assertEqual(repr(repeat('a', -2)), "repeat('a', 0)") | ||
| 973 | + self.assertEqual(repr(repeat('a', times=-1)), "repeat('a', 0)") | ||
| 974 | + self.assertEqual(repr(repeat('a', times=-2)), "repeat('a', 0)") | ||
| 975 | + | ||
| 970 | 976 | def test_map(self): | |
| 971 | 977 | self.assertEqual(list(map(operator.pow, range(3), range(1,7))), | |
| 972 | 978 | [0**1, 1**2, 2**3]) | |
@@ -1741,8 +1747,15 @@ class LengthTransparency(unittest.TestCase): | |||
| 1741 | 1747 | ||
| 1742 | 1748 | def test_repeat(self): | |
| 1743 | 1749 | self.assertEqual(operator.length_hint(repeat(None, 50)), 50) | |
| 1750 | + self.assertEqual(operator.length_hint(repeat(None, 0)), 0) | ||
| 1744 | 1751 | self.assertEqual(operator.length_hint(repeat(None), 12), 12) | |
| 1745 | 1752 | ||
| 1753 | + def test_repeat_with_negative_times(self): | ||
| 1754 | + self.assertEqual(operator.length_hint(repeat(None, -1)), 0) | ||
| 1755 | + self.assertEqual(operator.length_hint(repeat(None, -2)), 0) | ||
| 1756 | + self.assertEqual(operator.length_hint(repeat(None, times=-1)), 0) | ||
| 1757 | + self.assertEqual(operator.length_hint(repeat(None, times=-2)), 0) | ||
| 1758 | + | ||
| 1746 | 1759 | class RegressionTests(unittest.TestCase): | |
| 1747 | 1760 | ||
| 1748 | 1761 | def test_sf_793826(self): | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1388,6 +1388,7 @@ Norman Vine | |||
| 1388 | 1388 | Pauli Virtanen | |
| 1389 | 1389 | Frank Visser | |
| 1390 | 1390 | Johannes Vogel | |
| 1391 | + Vajrasky Kok | ||
| 1391 | 1392 | Alex Volkov | |
| 1392 | 1393 | Martijn Vries | |
| 1393 | 1394 | Sjoerd de Vries | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -31,6 +31,10 @@ Library | |||
| 31 | 31 | ||
| 32 | 32 | - Issue #21832: Require named tuple inputs to be exact strings. | |
| 33 | 33 | ||
| 34 | + - Issue #19145: The times argument for itertools.repeat now handles | ||
| 35 | + negative values the same way for keyword arguments as it does for | ||
| 36 | + positional arguments. | ||
| 37 | + | ||
| 34 | 38 | - Issue #21812: turtle.shapetransform did not tranform the turtle on the | |
| 35 | 39 | first call. (Issue identified and fixed by Lita Cho.) | |
| 36 | 40 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -4109,14 +4109,17 @@ repeat_new(PyTypeObject *type, PyObject *args, PyObject *kwds) | |||
| 4109 | 4109 | { | |
| 4110 | 4110 | repeatobject *ro; | |
| 4111 | 4111 | PyObject *element; | |
| 4112 | - Py_ssize_t cnt = -1; | ||
| 4112 | + Py_ssize_t cnt = -1, n_kwds = 0; | ||
| 4113 | 4113 | static char *kwargs[] = {"object", "times", NULL}; | |
| 4114 | 4114 | ||
| 4115 | 4115 | if (!PyArg_ParseTupleAndKeywords(args, kwds, "O|n:repeat", kwargs, | |
| 4116 | 4116 | &element, &cnt)) | |
| 4117 | 4117 | return NULL; | |
| 4118 | 4118 | ||
| 4119 | - if (PyTuple_Size(args) == 2 && cnt < 0) | ||
| 4119 | + if (kwds != NULL) | ||
| 4120 | + n_kwds = PyDict_Size(kwds); | ||
| 4121 | + /* Does user supply times argument? */ | ||
| 4122 | + if ((PyTuple_Size(args) + n_kwds == 2) && cnt < 0) | ||
| 4120 | 4123 | cnt = 0; | |
| 4121 | 4124 | ||
| 4122 | 4125 | ro = (repeatobject *)type->tp_alloc(type, 0); | |
| Back | FazBrowse Home | New Git URL |
0 commit comments