Skip to content

bpo-43770: _PyTypes_Init() inits _PyAnextAwaitable_Type #25266

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

Merged
merged 1 commit into from
Apr 8, 2021
Merged

bpo-43770: _PyTypes_Init() inits _PyAnextAwaitable_Type #25266

merged 1 commit into from
Apr 8, 2021

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Apr 7, 2021

  • Rename PyAnextAwaitable_Type to _PyAnextAwaitable_Type.
  • Expose the type in the internal C API.

https://bugs.python.org/issue43770

* Rename PyAnextAwaitable_Type to _PyAnextAwaitable_Type.
* Expose the type in the internal C API.
@@ -7,6 +7,9 @@ extern "C" {

PyAPI_DATA(PyTypeObject) PySeqIter_Type;
PyAPI_DATA(PyTypeObject) PyCallIter_Type;
#ifdef Py_BUILD_CORE
extern PyTypeObject _PyAnextAwaitable_Type;
#endif
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be moved inside _PyTypes_Init(). I tried to mimick other header files which defines all types.

@gvanrossum
Copy link
Member

+1

@vstinner vstinner merged commit b98eba5 into python:master Apr 8, 2021
@vstinner vstinner deleted the anext_await branch April 8, 2021 07:58
@vstinner
Copy link
Member Author

vstinner commented Apr 8, 2021

Merged. Thanks for the review @gvanrossum.

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.

4 participants