Skip to content

bpo-31238: pydoc ServerThread.stop() now joins itself#3151

Merged
vstinner merged 3 commits into
python:masterfrom
vstinner:pydoc_join
Aug 21, 2017
Merged

bpo-31238: pydoc ServerThread.stop() now joins itself#3151
vstinner merged 3 commits into
python:masterfrom
vstinner:pydoc_join

Conversation

@vstinner

@vstinner vstinner commented Aug 18, 2017

Copy link
Copy Markdown
Member

ServerThread.stop() now joins itself to wait until
DocServer.serve_until_quit() completes and then explicitly sets
its docserver attribute to None to break a reference cycle.

https://bugs.python.org/issue31238

ServerThread.stop() now joins itself to wait until
DocServer.serve_until_quit() completes and then explicitly sets
its docserver attribute to None to break a reference cycle.
@vstinner

Copy link
Copy Markdown
Member Author

cc @JulienPalard

@vstinner vstinner requested a review from vadmium August 18, 2017 23:52
@pitrou

pitrou commented Aug 21, 2017

Copy link
Copy Markdown
Member

Looks good to me.

@vstinner vstinner merged commit 4cab2cd into python:master Aug 21, 2017
@vstinner vstinner deleted the pydoc_join branch August 21, 2017 21:24
@vstinner

Copy link
Copy Markdown
Member Author

Ok, thanks for the review @pitrou.

GadgetSteve pushed a commit to GadgetSteve/cpython that referenced this pull request Sep 10, 2017
* bpo-31238: pydoc ServerThread.stop() now joins itself

ServerThread.stop() now joins itself to wait until
DocServer.serve_until_quit() completes and then explicitly sets
its docserver attribute to None to break a reference cycle.

* Add NEWS.d entry
vstinner added a commit that referenced this pull request Jun 1, 2018
)

ServerThread.stop() now joins itself to wait until
DocServer.serve_until_quit() completes and then explicitly sets
its docserver attribute to None to break a reference cycle.

(cherry picked from commit 4cab2cd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants