Skip to content

Conversation

@AZero13
Copy link

@AZero13 AZero13 commented Dec 8, 2025

Only so is locked, which leaves otherset vulnerable.

@AZero13 AZero13 requested a review from rhettinger as a code owner December 8, 2025 04:41
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@AZero13 AZero13 changed the title gh-142399 Lock both so and otherset when setting a symmetric difference gh-142399: Lock both so and otherset when setting a symmetric difference Dec 8, 2025
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

…ifference

Only so is locked, which leaves otherset vulnerable.
@bedevere-app
Copy link

bedevere-app bot commented Dec 8, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

}

Py_BEGIN_CRITICAL_SECTION(so);
Py_BEGIN_CRITICAL_SECTION2(so, otherset);
Copy link
Contributor

Choose a reason for hiding this comment

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

The other has just been created, so there are no other threads that have a reference. Is locking required then?

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.

2 participants