-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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-104469 : Convert _testcapi/vectorcall_limited.c to use AC #107857
Conversation
nahyeon-an
commented
Aug 11, 2023
•
edited by bedevere-bot
Loading
edited by bedevere-bot
- Issue: Improve the readability and maintainability of test_capi using the AC #104469
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Closing and re-opening to retrigger CLA checks. Sorry for the noise. |
|
|
|
@corona10 FYI this caused a few of the WebAssembly buildbots to fail, e.g., https://buildbot.python.org/all/#/builders/1046/builds/2731 . |
I will take a look, it can be easily fixed if the implicit function declarations is a real issue.
|
|
Okay, this is an issue between AC and limited C APIs. We should revert it first. Thank you @vstinner for investigation :) |
…se AC (pythongh-107857)" This reverts commit 2e27da1.
…se AC (pythongh-107857)" This reverts commit 2e27da1.
Right, see: |
Sounds good, go ahead with any needed reverts. It is not important that the _test* extension modules use clinic :) |
…se AC … (pythongh-107951) Revert "pythongh-104469 : Convert _testcapi/vectorcall_limited.c to use AC (pythongh-107857)" This reverts commit 2e27da1.
New attempt since Argument Clinic now supports the limited C API: PR #109691. |
@nahyeon-an: I merged again your PR as 8c521f0 (PR gh-109691). Thanks for your contribution! |