Skip to content

gh-133146: Add the old public get_value method to documentation and refactor code. - #133301

Merged
gpshead merged 8 commits into
python:mainfrom
YvesDup:sem-macosx-multiprocessing-get_value
Nov 29, 2025
Merged

gh-133146: Add the old public get_value method to documentation and refactor code.#133301
gpshead merged 8 commits into
python:mainfrom
YvesDup:sem-macosx-multiprocessing-get_value

Conversation

@YvesDup

@YvesDupYvesDup commented May 2, 2025

Copy link
Copy Markdown
Contributor
  1. Update documentation of multiprocessing.Lockclass.
  2. Refactor code to mask calls to the private _semlock attribute of multiprocessing.Sempahore class.
    The multiprocessing.BoundedSempahore, multiprocessing.Condition and multiprocessing.Queue classes will update.

📚 Documentation preview 📚: https://cpython-previews--133301.org.readthedocs.build/

Closes#133146

@YvesDup
YvesDup requested a review from gpshead as a code ownerMay 2, 2025 14:26
@YvesDupYvesDup changed the title gh-133146: Add the older public get_value method to documantation and refactor code.gh-133146: Add the old public get_value method to documantation and refactor code.May 2, 2025
@YvesDupYvesDup changed the title gh-133146: Add the old public get_value method to documantation and refactor code.gh-133146: Add the old public get_value method to documentation and refactor code.May 2, 2025
Comment threadLib/multiprocessing/synchronize.py
@YvesDup

YvesDup commented Jun 9, 2025

Copy link
Copy Markdown
ContributorAuthor

Why this method is undocumented ?
See here #84974

Updated: I only suggest to refactor code

@gpsheadgpshead added the docs Documentation in the Doc dir label Nov 29, 2025
don't describe documentation updates. just note that some internal details changed a bit.
@gpshead
gpshead disabled auto-merge November 29, 2025 21:42
@gpshead
gpshead enabled auto-merge (squash) November 29, 2025 21:43
@gpsheadgpshead added the needs backport to 3.14 bugs and security fixes label Nov 29, 2025
@gpshead
gpshead merged commit db098a4 into python:mainNov 29, 2025
48 checks passed
@github-project-automationgithub-project-automationBot moved this from Todo to Done in Docs PRsNov 29, 2025
@miss-islington-app

Copy link
Copy Markdown

Thanks @YvesDup for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@miss-islington-app

Copy link
Copy Markdown

Sorry, @YvesDup and @gpshead, I could not cleanly backport this to 3.14 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker db098a475a47b16d25c88d95dbcf0c6572c68576 3.14

hugovk pushed a commit to hugovk/cpython that referenced this pull request Nov 30, 2025
…mentation and refactor code. (pythonGH-133301)
also uses it within the internals in a few places.
(cherry picked from commit db098a4)
Co-authored-by: Duprat <yduprat@gmail.com>
@bedevere-app

Copy link
Copy Markdown

GH-142110 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Nov 30, 2025
gpshead pushed a commit that referenced this pull request Nov 30, 2025
…ion and refactor code. (GH-133301) (#142110)
also uses it within the internals in a few places.
(cherry picked from commit db098a4)
Co-authored-by: Duprat <yduprat@gmail.com>
@YvesDup

YvesDup commented Dec 1, 2025

Copy link
Copy Markdown
ContributorAuthor

Thank you @gpshead for approving this PR.

StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
…on and refactor code. (pythonGH-133301)
also uses it within the internals in a few places.
@YvesDup
YvesDup deleted the sem-macosx-multiprocessing-get_value branch February 9, 2026 20:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentation in the Doc dirskip news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Document the public multiprocessing.[Bounded]Semaphore.get_value method and refactor code

2 participants

@YvesDup@gpshead