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.

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

bpo-45123: PyAiter_Check and PyObject_GetAiter fix & rename. #28194

Merged
merged 2 commits into from Sep 7, 2021

Conversation

@1st1
Copy link
Member

@1st1 1st1 commented Sep 7, 2021

Fix PyAiter_Check to only check for the __anext__ presense (not for
__aiter__). Rename PyAiter_Check() to PyAIter_Check(),
PyObject_GetAiter() -> PyObject_GetAIter().

https://bugs.python.org/issue45123

Automerge-Triggered-By: GH:pablogsal

Fix PyAiter_Check to only check for the `__anext__` presense (not for
`__aiter__`). Rename `PyAiter_Check()` to `PyAIter_Check()`,
`PyObject_GetAiter()` -> `PyObject_GetAIter()`.
@1st1 1st1 requested review from gvanrossum and pablogsal Sep 7, 2021
@1st1 1st1 requested a review from python/windows-team as a code owner Sep 7, 2021
….8Eh9iI.rst

Co-authored-by: Dennis Sweeney <36520290+sweeneyde@users.noreply.github.com>
@pablogsal pablogsal merged commit 2c3474a into python:main Sep 7, 2021
12 checks passed
12 checks passed
@github-actions
Docs Docs
Details
@github-actions
Check for source changes
Details
@github-actions
Check if generated files are up to date
Details
@github-actions
Windows (x86)
Details
@github-actions
Windows (x64)
Details
@github-actions
macOS
Details
@github-actions
Ubuntu
Details
@github-actions
Ubuntu SSL tests with OpenSSL
Details
@github-actions
Address sanitizer Address sanitizer
Details
Azure Pipelines PR #20210907.13 succeeded
Details
@bedevere-bot
bedevere/issue-number Issue number 45123 found
Details
@bedevere-bot
bedevere/news News entry found in Misc/NEWS.d
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 7, 2021

Thanks @1st1 for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒🤖

miss-islington added a commit to miss-islington/cpython that referenced this pull request Sep 7, 2021
…H-28194)

Fix PyAiter_Check to only check for the `__anext__` presense (not for
`__aiter__`). Rename `PyAiter_Check()` to `PyAIter_Check()`,
`PyObject_GetAiter()` -> `PyObject_GetAIter()`.

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
(cherry picked from commit 2c3474a)

Co-authored-by: Yury Selivanov <yury@edgedb.com>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Sep 7, 2021

GH-28199 is a backport of this pull request to the 3.10 branch.

pablogsal added a commit to miss-islington/cpython that referenced this pull request Sep 7, 2021
…H-28194)

Fix PyAiter_Check to only check for the `__anext__` presense (not for
`__aiter__`). Rename `PyAiter_Check()` to `PyAIter_Check()`,
`PyObject_GetAiter()` -> `PyObject_GetAIter()`.

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
(cherry picked from commit 2c3474a)

Co-authored-by: Yury Selivanov <yury@edgedb.com>
pablogsal pushed a commit that referenced this pull request Sep 7, 2021
… (GH-28199)

Fix PyAiter_Check to only check for the `__anext__` presense (not for
`__aiter__`). Rename `PyAiter_Check()` to `PyAIter_Check()`,
`PyObject_GetAiter()` -> `PyObject_GetAIter()`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants