Skip to content

gh-106320: Remove _PyDict_GetItemStringWithError() function - #108313

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:testcapi
Aug 22, 2023
Merged

gh-106320: Remove _PyDict_GetItemStringWithError() function#108313
vstinner merged 1 commit into
python:mainfrom
vstinner:testcapi

Conversation

@vstinner

@vstinnervstinner commented Aug 22, 2023

Copy link
Copy Markdown
Member

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
Copy Markdown
MemberAuthor

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 :-(

@vstinnervstinner reopened this Aug 22, 2023
@vstinner
vstinner enabled auto-merge (squash) August 22, 2023 18:09
@vstinner
vstinner merged commit 615f6e9 into python:mainAug 22, 2023
@vstinner
vstinner deleted the testcapi branch August 22, 2023 18:19
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner@bedevere-bot