Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[3.10] bpo-42747: Remove Py_TPFLAGS_HAVE_AM_SEND and make Py_TPFLAGS_HAVE_VERSION_TAG no-op (GH-27260) #27306

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
Jul 23, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jul 23, 2021

  • Remove code that checks Py_TPFLAGS_HAVE_VERSION_TAG

    The field is always present in the type struct, as explained
    in the added comment.

  • Remove Py_TPFLAGS_HAVE_AM_SEND

    The flag is not needed, and since it was added in 3.10 it can be removed now.
    (cherry picked from commit a4760cc)

Co-authored-by: Petr Viktorin [email protected]

https://bugs.python.org/issue42747

…RSION_TAG no-op (pythonGH-27260)

* Remove code that checks Py_TPFLAGS_HAVE_VERSION_TAG

    The field is always present in the type struct, as explained
    in the added comment.

* Remove Py_TPFLAGS_HAVE_AM_SEND

    The flag is not needed, and since it was added in 3.10 it can be removed now.
(cherry picked from commit a4760cc)

Co-authored-by: Petr Viktorin <[email protected]>
@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

3 similar comments
@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

@encukou encukou merged commit 632e8a6 into python:3.10 Jul 23, 2021
@miss-islington miss-islington deleted the backport-a4760cc-3.10 branch July 23, 2021 14:57
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.

4 participants