Skip to content

Core: Add PyAwaitable_AddExpr for convenience#81

Merged
ZeroIntensity merged 3 commits intomasterfrom
add-expr
Aug 17, 2025
Merged

Core: Add PyAwaitable_AddExpr for convenience#81
ZeroIntensity merged 3 commits intomasterfrom
add-expr

Conversation

@ZeroIntensity
Copy link
Owner

Inspired by PyModule_Add. This function is equivalent to PyAwaitable_AddAwait, but returns NULL without setting an exception when given a NULL coroutine, and steals a reference to the coroutine when it is non-NULL. This makes it possible to directly use C API functions in the single PyAwaitable_AddExpr call.

@netlify
Copy link

netlify bot commented Aug 17, 2025

Deploy Preview for pyawaitable ready!

Name Link
🔨 Latest commit 44ebaca
🔍 Latest deploy log https://app.netlify.com/projects/pyawaitable/deploys/68a1fb28e6cf240008e2de68
😎 Deploy Preview https://deploy-preview-81--pyawaitable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ZeroIntensity ZeroIntensity changed the title Add PyAwaitable_AddExpr for convenience Core: Add PyAwaitable_AddExpr for convenience Aug 17, 2025
It seemed to break too many things to be testable.
@ZeroIntensity ZeroIntensity enabled auto-merge (squash) August 17, 2025 15:55
@ZeroIntensity ZeroIntensity merged commit 34dcee2 into master Aug 17, 2025
26 checks passed
@ZeroIntensity ZeroIntensity deleted the add-expr branch August 17, 2025 15:57
@AraHaan
Copy link
Collaborator

AraHaan commented Dec 14, 2025

This is a good addition to the API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants