← 返回首页
gh-93649: Split _testcapimodule.c into smaller files by vstinner · Pull Request #129400 · python/cpython · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted
Cancel Submit feedback

Saved searches

Use saved searches to filter your results more quickly

Cancel Create saved search
Appearance settings
Resetting focus

gh-93649: Split _testcapimodule.c into smaller files#129400

Closed
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:split_testcapi
Closed

gh-93649: Split _testcapimodule.c into smaller files#129400
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:split_testcapi

Conversation

vstinner commented Jan 28, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

  • Move many functions from _testcapimodule.c into more specific files in Modules/_testcapi/.

  • Add files:

    • Modules/_testcapi/frame.c
    • Modules/_testcapi/function.c
    • Modules/_testcapi/type.c

  • In moved code:

    • Replace get_testerror() with PyExc_AssertionError.
    • Replace raiseTestError() with PyErr_Format(PyExc_AssertionError, ...).

bedevere-app Bot mentioned this pull request Jan 28, 2025
Open
pythongh-93649: Split _testcapimodule.c into smaller files …
78d4204
* Move many functions from _testcapimodule.c into more specific files in Modules/_testcapi/. * Add files: * Modules/_testcapi/frame.c * Modules/_testcapi/function.c * Modules/_testcapi/type.c * In moved code: * Replace get_testerror() with PyExc_AssertionError. * Replace raiseTestError() with PyErr_Format(PyExc_AssertionError, ...).
vstinner marked this pull request as draft January 31, 2025 13:11

vstinner commented Feb 1, 2025

Copy link
Copy Markdown
Member Author

I have split this PR into smaller PRs:

vstinner closed this Feb 1, 2025
vstinner deleted the split_testcapi branch February 1, 2025 12:58
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

markshannon Awaiting requested review from markshannon markshannon is a code owner
erlend-aasland Awaiting requested review from erlend-aasland erlend-aasland is a code owner
ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently is a code owner
AA-Turner Awaiting requested review from AA-Turner AA-Turner will be requested when the pull request is marked ready for review AA-Turner is a code owner
emmatyping Awaiting requested review from emmatyping emmatyping will be requested when the pull request is marked ready for review emmatyping is a code owner
itamaro Awaiting requested review from itamaro itamaro will be requested when the pull request is marked ready for review itamaro is a code owner

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

Footer

© 2026 GitHub, Inc.