Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

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

Appearance settings

gh-134584: Revert partially #135860#142620

Merged
Fidget-Spinner merged 2 commits into
python:mainpython/cpython:mainfrom
Fidget-Spinner:partial_revert_135860Fidget-Spinner/cpython:partial_revert_135860Copy head branch name to clipboard
Dec 12, 2025
Merged

gh-134584: Revert partially #135860#142620
Fidget-Spinner merged 2 commits into
python:mainpython/cpython:mainfrom
Fidget-Spinner:partial_revert_135860Fidget-Spinner/cpython:partial_revert_135860Copy head branch name to clipboard

Conversation

@Fidget-Spinner

@Fidget-Spinner Fidget-Spinner commented Dec 12, 2025

Copy link
Copy Markdown
Member

@Zheaoli

Zheaoli commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

Should I change #136070 to avoid the refleak?

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM: I confirm that this change fix the refleak. I didn't review the change itself, it just looks good to me :-)

$ ./python -m test test.test_multiprocessing_fork.test_threads -R 3:3 
(...)
Result: SUCCESS

@Fidget-Spinner

Copy link
Copy Markdown
Member Author

Should I change #136070 to avoid the refleak?

The PR can't be fixed/ the code generator won't allow the change until #142604 is merged.

@Fidget-Spinner Fidget-Spinner merged commit a3a611b into python:main Dec 12, 2025
72 checks passed
@Fidget-Spinner Fidget-Spinner deleted the partial_revert_135860 branch December 12, 2025 14:05
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.