Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-121272: compiler no longer modifies the symtable, ste_coroutine always set during symtable construction#121297

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.

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

Already on GitHub?Sign in to your account

Merged
iritkatriel merged 1 commit intopython:mainfromiritkatriel:ste_coroutine
Jul 3, 2024

Conversation

@iritkatriel
Copy link
Member

@iritkatrieliritkatriel commentedJul 2, 2024
edited
Loading

compiler_exit_scope(c);
if (is_top_level_await&&is_async_generator){
c->u->u_ste->ste_coroutine=1;
assert(c->u->u_ste->ste_coroutine==1);
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

I believe this assignment has been obsolete for a while now, see#6766 (comment).

Copy link
Member

@JelleZijlstraJelleZijlstra left a comment

Choose a reason for hiding this comment

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

Agree this is a job for the symtable, not the compiler. This looks to be well tested intest_compile_top_level_await in test_builtin.py.

iritkatriel reacted with thumbs up emoji
@iritkatrieliritkatriel merged commit9315688 intopython:mainJul 3, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull requestJul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull requestJul 17, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@JelleZijlstraJelleZijlstraJelleZijlstra approved these changes

@carljmcarljmAwaiting requested review from carljmcarljm is a code owner

@markshannonmarkshannonAwaiting requested review from markshannonmarkshannon is a code owner

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@iritkatriel@JelleZijlstra

[8]ページ先頭

©2009-2025 Movatter.jp