Skip to content

use PyIterReturn for gen internal - #3183

Closed
youknowone wants to merge 2 commits into
RustPython:mainfrom
youknowone:iter-return
Closed

youknowone wants to merge 2 commits into
RustPython:mainfrom
youknowone:iter-return

Conversation

@youknowone

Copy link
Copy Markdown
Member

No description provided.

@youknowone
youknowone marked this pull request as draft October 1, 2021 19:38
@youknowone
youknowone force-pushed the iter-return branch 5 times, most recently from 7e39271 to f2f95fd Compare October 1, 2021 21:37
@youknowone
youknowone marked this pull request as ready for review October 1, 2021 21:38
Comment thread vm/src/coroutine.rs Outdated
pub struct Coro {
frame: FrameRef,
pub closed: AtomicCell<bool>,
pub closed: AtomicCell<bool>, // redudant?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If frame was made into an option closed could be removed. That's closer to what CPython does.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I replaced this comment to link to this comment

@DimitrisJim

Copy link
Copy Markdown
Member

merged #3185 first so other PRs that might come before this is merged won't get stuck on the failing tests.

@youknowone

Copy link
Copy Markdown
Member Author

merged by #3202

@youknowone
youknowone deleted the iter-return branch September 13, 2026 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants