Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make _symtable_entry.ste_type's comment consistent wit _Py_block_ty #92414

Merged
merged 1 commit into from Oct 7, 2022

Conversation

zikcheng
Copy link
Contributor

@zikcheng zikcheng commented May 7, 2022

_Py_block_ty defines four types of block, FunctionBlock, ClassBlock, ModuleBlock and AnnotationBlock.
But _symtable_entry.ste_type only comments three of them, I think it's better both sides are consistent.

_Py_block_ty defines four types of block, FunctionBlock, ClassBlock, ModuleBlock and AnnotationBlock.
But _symtable_entry.ste_type only comments three of them, I think it's better both sides are consistent.
@bedevere-bot
Copy link

bedevere-bot commented May 7, 2022

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

carljm
carljm approved these changes May 7, 2022
Copy link
Contributor

@carljm carljm left a comment

This changes only a code comment, so I don't think it needs an issue or news entry.

Comment change looks correct; thanks!

Copy link
Contributor

@kumaraditya303 kumaraditya303 left a comment

typedef enum _block_type { FunctionBlock, ClassBlock, ModuleBlock, AnnotationBlock }
    _Py_block_ty;

@JelleZijlstra JelleZijlstra merged commit 24a4b34 into python:main Oct 7, 2022
12 checks passed
@zikcheng zikcheng deleted the fix_comment branch Oct 8, 2022
mpage pushed a commit to mpage/cpython that referenced this pull request Oct 11, 2022
…ython#92414)

_Py_block_ty defines four types of block, FunctionBlock, ClassBlock, ModuleBlock and AnnotationBlock.
But _symtable_entry.ste_type only comments three of them, I think it's better both sides are consistent.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants