Visitar URL original
gh-101892: Fix SystemError when a callable iterator call exhausts the iterator by workingpayload · Pull Request #101896 · python/cpython · GitHub
Skip to content

gh-101892: Fix SystemError when a callable iterator call exhausts the iterator #101896

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kumaraditya303 merged 19 commits into from
Mar 4, 2023
Merged

gh-101892: Fix SystemError when a callable iterator call exhausts the iterator #101896

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
de17a82
gh-101892 : Updated Cython/objects/iterobject.c Line 217
workingpayload Feb 14, 2023
3fba831
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 14, 2023
bee2af7
Clarify the news entry
arhadthedev Feb 14, 2023
25f4c06
Merge branch 'main' into workingpayload/issue101892
workingpayload Feb 14, 2023
0c631e3
Add a test
arhadthedev Feb 14, 2023
506fb6a
Amend wording
arhadthedev Feb 14, 2023
78168bf
Rep
arhadthedev Feb 15, 2023
188e388
Address the review
arhadthedev Feb 15, 2023
0f9cea1
gh-101892 : Updated Cython/objects/iterobject.c Line 217
workingpayload Feb 15, 2023
6abb09a
Merge branch 'main' into workingpayload/issue101892
workingpayload Feb 15, 2023
c8b6bad
Update test_iter.py
arhadthedev Feb 15, 2023
6931c20
Remove an unnecessary abstraction of reentrance
arhadthedev Feb 15, 2023
36b043c
Clarify comments once more
arhadthedev Feb 15, 2023
340f668
Update 2023-02-14-09-08-48.gh-issue-101892.FMos8l.rst
arhadthedev Feb 15, 2023
fe1502d
Fix issues reported by patchcheck
arhadthedev Feb 15, 2023
b9065d9
Merge branch 'main' into workingpayload/issue101892
arhadthedev Mar 4, 2023
cc2066b
Do not nitpick for empty lines of others
arhadthedev Mar 4, 2023
27e478f
Fix mismerging: move the new method
arhadthedev Mar 4, 2023
1ee2293
Address the review
arhadthedev Mar 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix issues reported by patchcheck
  • Loading branch information
@arhadthedev
arhadthedev committed Feb 15, 2023
commit fe1502dab4d2c60a68d413138e0d4a55a42c8222
6 changes: 3 additions & 3 deletions Lib/test/test_iter.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ def __call__(self):

def exhaust(iterator):
Comment thread
arhadthedev marked this conversation as resolved.
Outdated
"""Exhaust an iterator without raising StopIteration."""
list(iterator)
list(iterator)


# Main test suite
Expand Down Expand Up @@ -273,7 +273,7 @@ def spam(state=[0]):
return i
self.check_iterator(iter(spam, 20), list(range(10)), pickle=False)

# Test two-argument iter() with function that exhausts its
# Test two-argument iter() with function that exhausts its
Comment thread
arhadthedev marked this conversation as resolved.
Outdated
# associated iterator but forgets to either return a sentinel value
# or raise `StopIteration`.
def test_iter_function_concealing_reentrant_exhaustion(self):
Expand All @@ -287,7 +287,7 @@ def spam():
spam.is_recursive_call = True
exhaust(spam.iterator)
return HAS_MORE

spam.is_recursive_call = False
spam.iterator = iter(spam, NO_MORE)
with self.assertRaises(StopIteration):
Expand Down