Skip to content

[3.6] bpo-31965: fix doc for multiprocessing.connection.Client and Listener (GH-4304)#4321

Merged
pitrou merged 1 commit into
python:3.6from
miss-islington:backport-1e5d54c-3.6
Nov 7, 2017
Merged

[3.6] bpo-31965: fix doc for multiprocessing.connection.Client and Listener (GH-4304)#4321
pitrou merged 1 commit into
python:3.6from
miss-islington:backport-1e5d54c-3.6

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Nov 7, 2017

  • fix doc for multiprocessing.connection.Client

The authenticate argument does not exist on either Client or Listener:

Also made some small changes to vary sentence structure.
(cherry picked from commit 1e5d54c)

https://bugs.python.org/issue31965

…pythonGH-4304)

* fix doc for multiprocessing.connection.Client

The authenticate argument does not exist on either Client or Listener:
- https://github.com/python/cpython/blob/master/Lib/multiprocessing/connection.pyGH-L483 (master)
- https://github.com/python/cpython/blob/3.6/Lib/multiprocessing/connection.pyGH-L478 (3.6)
- https://github.com/python/cpython/blob/3.5/Lib/multiprocessing/connection.pyGH-L478 (3.5)
- https://github.com/python/cpython/blob/3.4/Lib/multiprocessing/connection.pyGH-L487 (3.4)
The documentation also claimed that these functions will call `current_process().auth_key`, for which I could find no evidence in the code. I rewrote the documentation to reflect the actual behavior.

Also made some small changes to vary sentence structure.
(cherry picked from commit 1e5d54c)
@miss-islington
Copy link
Copy Markdown
Contributor Author

@JelleZijlstra and @pitrou: Backport status check is done, and it's a success ✅ .

@pitrou pitrou merged commit d9c61c2 into python:3.6 Nov 7, 2017
@miss-islington
Copy link
Copy Markdown
Contributor Author

Thanks, @pitrou!

@miss-islington miss-islington deleted the backport-1e5d54c-3.6 branch November 7, 2017 16:20
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.

5 participants