Skip to content

gh-155742: Use PyBytesWriter in Python/assemble.c - #155747

Merged
vstinner merged 3 commits into
python:mainfrom
vstinner:assemble
Aug 14, 2026
Merged

gh-155742: Use PyBytesWriter in Python/assemble.c#155747
vstinner merged 3 commits into
python:mainfrom
vstinner:assemble

Conversation

@vstinner

@vstinner vstinner commented Aug 13, 2026

Copy link
Copy Markdown
Member

Replace soft deprecated PyBytes_FromStringAndSize() with PyBytesWriter.

Replace soft deprecated PyBytes_FromStringAndSize() with
PyBytesWriter.
@vstinner
vstinner requested a review from markshannon as a code owner August 13, 2026 18:02
@vstinner vstinner added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Aug 13, 2026
@vstinner
vstinner requested a review from iritkatriel as a code owner August 13, 2026 18:02
Comment thread Python/assemble.c Outdated
Comment thread Python/assemble.c Outdated
* Rename a_linetable member to a_linetable_writer
* Rename a_linetable_obj member to a_linetable
* Rename a_linetable variable to linetable
@vstinner

Copy link
Copy Markdown
Member Author

@iritkatriel: Please review the updated PR. I renamed variables.

@iritkatriel

Copy link
Copy Markdown
Member

I'd run a ref leaks test just in case.

@vstinner
vstinner enabled auto-merge (squash) August 14, 2026 15:37
@vstinner

Copy link
Copy Markdown
Member Author

I enabled auto-merge. Thanks for the review!

@vstinner
vstinner merged commit 09b6317 into python:main Aug 14, 2026
51 checks passed
@vstinner
vstinner deleted the assemble branch August 14, 2026 16:09
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants