[issue20769] Reload() description is unclear
Robert Collins added the comment: 2.7 side applied. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Roundup Robot added the comment: New changeset cfd768814ca3 by Robert Collins in branch '2.7': Issue #20769: Improve reload() docs. Patch by Dorian Pula. https://hg.python.org/cpython/rev/cfd768814ca3 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Robert Collins added the comment: 24081 invalidates half the patch, but the other half still applies. -- nosy: +rbcollins versions: +Python 3.4, Python 3.6 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Changes by Arnon Yaari wiggi...@gmail.com: -- nosy: +wiggin15 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Brett Cannon added the comment: Patch LGTM. -- stage: - commit review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Changes by Dorian Pula dorian.p...@amber-penguin-software.ca: -- keywords: +patch versions: +Python 3.5 Added file: http://bugs.python.org/file39019/reload_importlib_doc_py_3_5_fix.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Dorian Pula added the comment: Attached patches with proposed change to the documentation to make the description clearer. Please review and comment. -- nosy: +dorianpula ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Changes by Dorian Pula dorian.p...@amber-penguin-software.ca: Added file: http://bugs.python.org/file39020/reload_builtin_functions_doc_py_2_7_fix.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
New submission from Roy Smith: http://docs.python.org/2/library/functions.html#reload says: It is legal though generally not very useful to reload built-in or dynamically loaded modules, except for sys, __main__ and __builtin__. It is unclear what the except for ... part is referring to. Is it not legal to reload those modules? Or is it not very useful to reload them? -- assignee: docs@python components: Documentation messages: 212187 nosy: docs@python, roysmith priority: normal severity: normal status: open title: Reload() description is unclear versions: Python 2.7 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
R. David Murray added the comment: The python3 docs say: It is legal though generally not very useful to reload built-in or dynamically loaded modules (this is not true for e.g. sys, __main__, builtins and other key modules where reloading is frowned upon). So, it is the former...sort of. You don't get an error when you reload them, so implying that it is not legal is an odd phrasing. Probably that sentence should be clarified in both the python2 and python3 docs. -- nosy: +brett.cannon, eric.smith, r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
Changes by R. David Murray rdmur...@bitdance.com: -- Removed message: http://bugs.python.org/msg212197 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20769] Reload() description is unclear
R. David Murray added the comment: The python3 docs say: It is legal though generally not very useful to reload built-in or dynamically loaded modules (this is not true for e.g. sys, __main__, builtins and other key modules where reloading is frowned upon). So, it is the former...sort of. You don't get an error when you reload them, so implying that it is not legal is an odd phrasing. Probably that sentence should be clarified in both the python2 and python3 docs. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20769 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com