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.

Sign up for GitHub

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

Jump to bottom

gh-116417: Fix make check-c-globals for _testlimitedcapi #116570

Merged
merged 1 commit into from Mar 10, 2024
Merged

gh-116417: Fix make check-c-globals for _testlimitedcapi #116570

merged 1 commit into from Mar 10, 2024

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Mar 10, 2024 •
edited by bedevere-app bot

  • Remove unused '_testcapimodule' global in Modules/_testcapi/unicode.c.
  • Update c-analyzer to not use the internal C API in _testlimitedcapi.c.

@vstinner
pythongh-116417: Fix make check-c-globals for _testlimitedcapi
256fdc6 
* Remove unused '_testcapimodule' global in Modules/_testcapi/unicode.c.
* Update c-analyzer to not use the internal C API in
  _testlimitedcapi.c.
@bedevere-app bedevere-app bot mentioned this pull request Mar 10, 2024
Closed
@vstinner vstinner enabled auto-merge (squash) March 10, 2024 20:01
@vstinner vstinner merged commit c5fa796 into python:main Mar 10, 2024
34 checks passed
@vstinner vstinner deleted the limited_globals branch March 10, 2024 20:19
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
@vstinner @adorilson
pythongh-116417: Fix make check-c-globals for _testlimitedcapi (pytho…
ca8b8bb 
…n#116570)

* Remove unused '_testcapimodule' global in Modules/_testcapi/unicode.c.
* Update c-analyzer to not use the internal C API in
  _testlimitedcapi.c.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
@vstinner @diegorusso
pythongh-116417: Fix make check-c-globals for _testlimitedcapi (pytho…
56b307f 
…n#116570)

* Remove unused '_testcapimodule' global in Modules/_testcapi/unicode.c.
* Update c-analyzer to not use the internal C API in
  _testlimitedcapi.c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently 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.

None yet
1 participant
@vstinner