Skip to content

bpo-43908: Add Py_TPFLAGS_IMMUTABLETYPE flag#25520

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 10 commits into from
Apr 28, 2021
Merged

bpo-43908: Add Py_TPFLAGS_IMMUTABLETYPE flag #25520

vstinner merged 10 commits into from
Apr 28, 2021

Conversation

@erlend-aasland
Copy link
Contributor

@erlend-aaslanderlend-aasland commented Apr 22, 2021•
edited by bedevere-bot
Loading

@erlend-aaslanderlend-aasland marked this pull request as ready for review April 22, 2021 11:50
gvanrossum
gvanrossum reviewed Apr 24, 2021
View reviewed changes
shreyanavigyan
shreyanavigyan reviewed Apr 26, 2021
View reviewed changes
@erlend-aaslanderlend-aasland marked this pull request as draft April 26, 2021 08:22
@erlend-aaslanderlend-aasland changed the title bpo-43908: Add Py_TPFLAGS_IMMUTABLE flag and use it in arraymodulebpo-43908: Add Py_TPFLAGS_IMMUTABLE flagApr 26, 2021
@erlend-aaslanderlend-aasland marked this pull request as ready for review April 26, 2021 18:50
@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Apr 26, 2021

cc. @pablogsal@vstinner@serhiy-storchaka@tiran

@erlend-aaslanderlend-aasland changed the title bpo-43908: Add Py_TPFLAGS_IMMUTABLE flagbpo-43908: Add Py_TPFLAGS_IMMUTABLETYPE flagApr 26, 2021
pablogsal
pablogsal reviewed Apr 26, 2021
View reviewed changes
@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Apr 27, 2021

@pablogsal are you ok with the PR as it is?

vstinner
vstinner reviewed Apr 28, 2021
View reviewed changes
Erlend Egeberg Aaslandand others added 3 commits April 28, 2021 17:16
@vstinner
Improve docs
025af5b
Co-authored-by: Victor Stinner <[email protected]>
@vstinner
Improve docs
0e8c709
Co-authored-by: Victor Stinner <[email protected]>
@vstinner
Improve comment
8c79eae
Co-authored-by: Victor Stinner <[email protected]>
vstinner
vstinner reviewed Apr 28, 2021
View reviewed changes
Erlend Egeberg Aaslandand others added 2 commits April 28, 2021 17:37
vstinner
vstinner approved these changes Apr 28, 2021
View reviewed changes
Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

IMO it was important to mention "type attributes cannot be set nor deleted" to explain what "immutable" means. I stop nitpicking, let's move on to add this flag to types recently converted to heap types.

@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Apr 28, 2021

Thanks, @vstinner! I appreciate the doc/comment nitpicking; you're improved the PR considerably.

@vstinner
Copy link
Member

vstinner commented Apr 28, 2021

@erlend-aasland: Travis CI failed:

Fixing docs whitespace ... 1 file: Doc/c-api/typeobj.rst Please fix the 1 file(s) with whitespace issues (on UNIX you can run `make patchcheck` to make the fixes)

shreyanavigyan
shreyanavigyan reviewed Apr 28, 2021
View reviewed changes
@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Apr 28, 2021

@erlend-aasland: Travis CI failed:

Oops, I'll fix it. I normally do make html in the Doc directory to make sure I've got the reST markup right.

@erlend-aasland
Copy link
ContributorAuthor

erlend-aasland commented Apr 28, 2021

FYI, unrelated random ssl test failure on the macOS CI, @vstinner.

@vstinnervstinner merged commit 3b52c8d into python:masterApr 28, 2021
@vstinner
Copy link
Member

vstinner commented Apr 28, 2021

Merged, thanks.

@erlend-aaslanderlend-aasland deleted the immutable-types branch April 28, 2021 17:08
@pablogsal
Copy link
Member

pablogsal commented Apr 28, 2021•
edited
Loading

FYI, unrelated random ssl test failure on the macOS CI, @vstinner.

CC: @tiran This is happening white a lot

@gvanrossumgvanrossum mentioned this pull request May 18, 2022
Closed
@maxbachmannmaxbachmann mentioned this pull request Sep 12, 2022
Merged
da-woods pushed a commit to cython/cython that referenced this pull request Jan 1, 2023
@maxbachmann
Mark extension types as immutable (#5023)
5e5b242
Since python/cpython#25520 types are automatically marked as immutable if they are static. While we still have the Py_TPFLAGS_HEAPTYPE hack in place we need to manually mark our types as immutable. It's possible to crash Python by trying to modify mutable static types.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pablogsalpablogsalpablogsal left review comments

@vstinnervstinnervstinner approved these changes

@markshannonmarkshannonAwaiting requested review from markshannon

@gvanrossumgvanrossumAwaiting requested review from gvanrossum

+1 more reviewer

@shreyanavigyanshreyanavigyanshreyanavigyan left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@erlend-aasland@vstinner@pablogsal@gvanrossum@shreyanavigyan@the-knights-who-say-ni@bedevere-bot