◐ Shell
clean mode source ↗

bpo-46541: Remove usage of _Py_IDENTIFIER from multibytecodec by corona10 · Pull Request #31475 · python/cpython

Skip to content

Navigation Menu

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

corona10

merged 3 commits into

Mar 1, 2022

Conversation

erlend-aasland

@terryjreedy

In the issue number, 65 was transposed as 56. Fixed.

@terryjreedy terryjreedy changed the title bpo-45641: Remove usage of _Py_IDENTIFIER from multibytecodec bpo-46541: Remove usage of _Py_IDENTIFIER from multibytecodec

Feb 21, 2022

erlend-aasland

kumaraditya303

Choose a reason for hiding this comment

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

LGTM

tiran

@bedevere-bot

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@corona10

I have made the requested changes; please review again

@bedevere-bot

Thanks for making the requested changes!

@tiran: please review the changes made to this pull request.

@corona10

After some discussion, I decided to merge this PR anyway.
I will accept the other approach if someone wants :) I hope that this approach would help to reduce the usage of private APIs
cc @ericsnowcurrently, @erlend-aasland, @tiran, @vstinner (alphabetical order :)

@corona10 corona10 deleted the bpo-46541-multibytecodec branch

March 1, 2022 14:35

Labels