FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.11] gh-105375: Explicitly initialise all {Pickler,Unpickler}Object fields (#105686) by erlend-aasland · Pull Request #105711 · python/cpython · GitHub

Repository navigation

[3.11] gh-105375: Explicitly initialise all {Pickler,Unpickler}Object fields (#105686) - #105711

Merged
kumaraditya303 merged 2 commits into
python:3.11from
erlend-aasland:backport-3.11-fix-pickle-newgcobj
Jun 13, 2023
Merged

kumaraditya303 merged 2 commits into
python:3.11from
erlend-aasland:backport-3.11-fix-pickle-newgcobj

Conversation

erlend-aasland commented Jun 12, 2023 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

All fields must be explicitly initialised to prevent manipulation of
uninitialised fields in dealloc.

Align initialisation order with the layout of the object structs.

(cherry picked from commit ca3cc4b)

Co-authored-by: Erlend E. Aasland erlend.aasland@protonmail.com

…Object fields (python#105686)

All fields must be explicitly initialised to prevent manipulation of
uninitialised fields in dealloc.

Align initialisation order with the layout of the object structs.

(cherry picked from commit ca3cc4b)

Co-authored-by: Erlend E. Aasland <erlend.aasland@protonmail.com>
kumaraditya303 disabled auto-merge June 13, 2023 06:24
kumaraditya303 enabled auto-merge (squash) June 13, 2023 06:24

Copy link
Copy Markdown
Contributor

Retrigged CI

kumaraditya303 merged commit 36c393c into python:3.11 Jun 13, 2023
erlend-aasland deleted the backport-3.11-fix-pickle-newgcobj branch June 13, 2023 07:05

Copy link
Copy Markdown
Contributor Author

Thanks, Kumar!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL