Skip to content

gh-107149: make new opcode util functions private rather than public and unstable#112042

Merged
iritkatriel merged 6 commits intopython:mainfrom
iritkatriel:opcode_utils
Nov 14, 2023
Merged

gh-107149: make new opcode util functions private rather than public and unstable#112042
iritkatriel merged 6 commits intopython:mainfrom
iritkatriel:opcode_utils

Conversation

@iritkatriel
Copy link
Copy Markdown
Member

@iritkatriel iritkatriel commented Nov 13, 2023

Comment thread Include/cpython/compile.h Outdated
@vstinner
Copy link
Copy Markdown
Member

vstinner added the skip news label Nov 13, 2023

Since functions are new in Python 3.13, there is no need to document these changes.

Comment thread Include/cpython/compile.h Outdated
Comment thread Include/cpython/compile.h Outdated
Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for moving definitions to pycore_compile.h.

Comment thread Include/cpython/compile.h Outdated
@iritkatriel iritkatriel enabled auto-merge (squash) November 14, 2023 00:09
@iritkatriel iritkatriel merged commit 36aab34 into python:main Nov 14, 2023
@vstinner
Copy link
Copy Markdown
Member

Thanks @iritkatriel. It's better to keep the API private for now, and only consider making it public if we have a clear user request to make it public (even if it's as an "unstable" API).

aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

interpreter-core (Objects, Python, Grammar, and Parser dirs) skip news topic-C-API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants