Skip to content
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

gh-106320: Remove _PyDict_GetItemStringWithError() function #108313

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Aug 22, 2023

Remove private _PyDict_GetItemStringWithError() function of the public C API: the new PyDict_GetItemStringRef() can be used instead.

  • Move private _PyDict_GetItemStringWithError() to the internal C API.
  • _testcapi get_code_extra_index() uses PyDict_GetItemStringRef(). Avoid using private functions in _testcapi which tests the public C API.

Remove private _PyDict_GetItemStringWithError() function of the
public C API: the new PyDict_GetItemStringRef() can be used instead.

* Move private _PyDict_GetItemStringWithError() to the internal C API.
* _testcapi get_code_extra_index() uses PyDict_GetItemStringRef().
  Avoid using private functions in _testcapi which tests the public C
  API.
@vstinner
Copy link
Member Author

Oh. While all tests passed, "Tests / All required checks pass (pull_request)" job is stuck at Queued — Waiting to run this check :-( The Docs job was skipped.

I close/reopen the PR to re-trigger all CI jobs :-(

@vstinner vstinner closed this Aug 22, 2023
@vstinner vstinner reopened this Aug 22, 2023
@vstinner vstinner enabled auto-merge (squash) August 22, 2023 18:09
@vstinner vstinner merged commit 615f6e9 into python:main Aug 22, 2023
32 checks passed
@vstinner vstinner deleted the testcapi branch August 22, 2023 18:19
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.

2 participants