Skip to content

Unused variables in mutliprocessing.managers.Server #97816

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.

Sign up for GitHub

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

Jump to bottom
Closed
saito828koki opened this issue Oct 4, 2022 · 1 comment
Closed

Unused variables in mutliprocessing.managers.Server #97816

saito828koki opened this issue Oct 4, 2022 · 1 comment
Labels
sprint

Comments

@saito828koki
Copy link
Contributor

The following 3 variables obj, exposed and gettypeid are declared but not used. We can delete this line.

cpython/Lib/multiprocessing/managers.py

Line 436 in 72b5a55

obj, exposed, gettypeid = self.id_to_obj[ident]

@bedevere-bot bedevere-bot mentioned this issue Oct 4, 2022
Merged
gpshead pushed a commit that referenced this issue Oct 4, 2022
@saito828koki
gh-97816: Remove unused variables in mutliprocessing.managers.Server (
27e59af 
#97817)

Remove unused local variables.
@gpshead
Copy link
Member

gpshead commented Oct 4, 2022

odd, perhaps an old line for extra debugging? anyways obviously unnecessary. thanks!

@gpshead gpshead closed this as completed Oct 4, 2022
@gpshead gpshead added the sprint label Oct 4, 2022
@ezio-melotti ezio-melotti moved this to Todo in Sprint 2024 Oct 4, 2022
@terryjreedy terryjreedy moved this from Todo to Done in Sprint 2024 Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
sprint
Projects
Sprint 2024
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gpshead @saito828koki