Skip to content

bpo-41798: Allocate _decimal extension module CAPI on the heap #24117

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.

Sign up for GitHub

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

Jump to bottom
Merged
vstinner merged 5 commits into from
Jan 6, 2021
Merged

bpo-41798: Allocate _decimal extension module CAPI on the heap #24117

vstinner merged 5 commits into from
Jan 6, 2021

Conversation

@erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Jan 5, 2021 •
edited by bedevere-bot
Loading

@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented Jan 5, 2021

cc. @shihai1991

shihai1991
shihai1991 reviewed Jan 5, 2021
View reviewed changes
@shihai1991 shihai1991 requested a review from vstinner January 5, 2021 15:54
shihai1991
shihai1991 reviewed Jan 5, 2021
View reviewed changes
@erlend-aasland
Copy link
Contributor Author

erlend-aasland commented Jan 5, 2021

Thanks for your review, @shihai1991! PTAL

@vstinner vstinner merged commit fe9f446 into python:master Jan 6, 2021
@vstinner
Copy link
Member

vstinner commented Jan 6, 2021

Merged, thanks.

@erlend-aasland erlend-aasland deleted the bpo-41798/decimal branch January 6, 2021 11:48
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
@adorilson
bpo-41798: Allocate _decimal extension module C API on the heap (pyth…
94f8545 
…onGH-24117)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rhettinger rhettinger Awaiting requested review from rhettinger

@vstinner vstinner Awaiting requested review from vstinner

@shihai1991 shihai1991 Awaiting requested review from shihai1991

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@erlend-aasland @vstinner @shihai1991 @the-knights-who-say-ni @bedevere-bot