summaryrefslogtreecommitdiffstats
path: root/Modules
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2016-03-30 18:11:16 (GMT)
committerSerhiy Storchaka <storchaka@gmail.com>2016-03-30 18:11:16 (GMT)
commitab0d198c7a6b2133ba5f782d7b951fef00ae615c (patch)
treea06cc9e22092d9e4cbb0c7bc5aaf96d5eac72ab6 /Modules
parentf39c0ac62f90842b790a41f1b0ddd51a6e3d8dc2 (diff)
downloadcpython-ab0d198c7a6b2133ba5f782d7b951fef00ae615c.zip
cpython-ab0d198c7a6b2133ba5f782d7b951fef00ae615c.tar.gz
cpython-ab0d198c7a6b2133ba5f782d7b951fef00ae615c.tar.bz2
Issue #26492: Exhausted iterator of array.array now conforms with the behavior
of iterators of other mutable sequences: it lefts exhausted even if iterated array is extended.
Diffstat (limited to 'Modules')
-rw-r--r--Modules/arraymodule.c22
1 files changed, 18 insertions, 4 deletions
diff --git a/Modules/arraymodule.c b/Modules/arraymodule.c
index 1b0a282..323e0c1 100644
--- a/Modules/arraymodule.c
+++ b/Modules/arraymodule.c
@@ -2875,9 +2875,20 @@ array_iter(arrayobject *ao)
static PyObject *
arrayiter_next(arrayiterobject *it)
{
+ arrayobject *ao;
+
+ assert(it != NULL);
assert(PyArrayIter_Check(it));
- if (it->index < Py_SIZE(it->ao))
- return (*it->getitem)(it->ao, it->index++);
+ ao = it->ao;
+ if (ao == NULL) {
+ return NULL;
+ }
+ assert(array_Check(ao));
+ if (it->index < Py_SIZE(ao)) {
+ return (*it->getitem)(ao, it->index++);
+ }
+ it->ao = NULL;
+ Py_DECREF(ao);
return NULL;
}
@@ -2906,8 +2917,11 @@ static PyObject *
array_arrayiterator___reduce___impl(arrayiterobject *self)
/*[clinic end generated code: output=7898a52e8e66e016 input=a062ea1e9951417a]*/
{
- return Py_BuildValue("N(O)n", _PyObject_GetBuiltin("iter"),
- self->ao, self->index);
+ PyObject *func = _PyObject_GetBuiltin("iter");
+ if (self->ao == NULL) {
+ return Py_BuildValue("N(())", func);
+ }
+ return Py_BuildValue("N(O)n", func, self->ao, self->index);
}
/*[clinic input]