◐ Shell
clean mode source ↗

Revert "gh-115432: Add critical section variant that handles a NULL object (#115433)" by colesbury · Pull Request #118861 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@colesbury

@colesbury colesbury commented

May 9, 2024

edited by bedevere-app Bot

Loading

Copy link Copy Markdown

Contributor

This reverts commit ad4f909.

This internal API ended up not being used.

…NULL object (python#115433)"

This reverts commit ad4f909.

The API ended up not being used.

@colesbury

Copy link Copy Markdown

Contributor Author

@Yhg1s - this isn't really a bug fix, but it's definitely not a feature. Would it be appropriate to backport this revert to 3.13?

@colesbury colesbury force-pushed the gh-115432-revert branch from 9f8175b to 49a359e Compare

May 9, 2024 18:26

@DinoV DinoV left a comment

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

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

LGTM!

@Yhg1s

Yhg1s commented

May 9, 2024

Copy link Copy Markdown

Member

Yeah, backporting it to 3.13 is fine until we get to rc1.

colesbury reacted with thumbs up emoji

@colesbury colesbury added the needs backport to 3.13

bugs and security fixes

label

May 9, 2024

@colesbury colesbury merged commit 46c8081 into python:main

May 9, 2024

@colesbury colesbury deleted the gh-115432-revert branch

May 9, 2024 22:33

@miss-islington-app

Copy link Copy Markdown

Thanks @colesbury for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

May 9, 2024
…NULL object (pythonGH-115433)" (pythonGH-118861)

This reverts commit ad4f909.

The API ended up not being used.
(cherry picked from commit 46c8081)

Co-authored-by: Sam Gross <colesbury@gmail.com>

@bedevere-app

bedevere-app Bot commented

May 9, 2024

Copy link Copy Markdown

GH-118872 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13

bugs and security fixes

label

May 9, 2024

colesbury added a commit that referenced this pull request

May 9, 2024
… NULL object (GH-115433)" (GH-118861) (#118872)

This reverts commit ad4f909.

The API ended up not being used.
(cherry picked from commit 46c8081)

Co-authored-by: Sam Gross <colesbury@gmail.com>

SonicField pushed a commit to SonicField/cpython that referenced this pull request

May 10, 2024
…NULL object (python#115433)" (python#118861)

This reverts commit ad4f909.

The API ended up not being used.

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

Reviewers

@DinoV DinoV DinoV approved these changes

@Yhg1s Yhg1s Awaiting requested review from Yhg1s

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@colesbury @Yhg1s @DinoV