Skip to content

GH-131238: Core header refactor #131250

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.

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

Merged
merged 12 commits into from
Mar 17, 2025

Conversation

markshannon
Copy link
Member

@markshannon markshannon commented Mar 14, 2025

This PR moves most of the struct definitions in the various "core" header files to new header files.
This breaks the dependencies between header files, as headers do not need to include each other, but can include one or two of the struct definition headers.

@markshannon markshannon added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Mar 14, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @markshannon for commit 4231362 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F131250%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

Please add also the 3 new internal headers to PCbuild/pythoncore.vcxproj and PCbuild/pythoncore.vcxproj.filters.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM. I like the overall design. I just left a few minor comments which can be addressed later if needed (I can handle them if you want).

@markshannon markshannon merged commit a1aeec6 into python:main Mar 17, 2025
41 checks passed
@markshannon markshannon deleted the core-header-refactor branch March 17, 2025 09:19
@markshannon markshannon restored the core-header-refactor branch March 17, 2025 11:26
@markshannon markshannon deleted the core-header-refactor branch March 17, 2025 11:27
plashchynski pushed a commit to plashchynski/cpython that referenced this pull request Mar 17, 2025
* Moves most structs in pycore_ header files into pycore_structs.h and pycore_runtime_structs.h

* Removes many cross-header dependencies
seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
* Moves most structs in pycore_ header files into pycore_structs.h and pycore_runtime_structs.h

* Removes many cross-header dependencies
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.

4 participants