Issue #26844: Fix imp.find_module() to have the exception related to

type issues be about 'path' instead of 'name'.

Thanks to Lev Maximov for the patch.
This commit is contained in:
Brett Cannon 2016-07-15 10:58:54 -07:00
parent 5d9c7ed55a
commit f76457e122
3 changed files with 6 additions and 2 deletions

View File

@ -266,8 +266,8 @@ def find_module(name, path=None):
raise TypeError("'name' must be a str, not {}".format(type(name))) raise TypeError("'name' must be a str, not {}".format(type(name)))
elif not isinstance(path, (type(None), list)): elif not isinstance(path, (type(None), list)):
# Backwards-compatibility # Backwards-compatibility
raise RuntimeError("'list' must be None or a list, " raise RuntimeError("'path' must be None or a list, "
"not {}".format(type(name))) "not {}".format(type(path)))
if path is None: if path is None:
if is_builtin(name): if is_builtin(name):

View File

@ -939,6 +939,7 @@ Graham Matthews
mattip mattip
Martin Matusiak Martin Matusiak
Dieter Maurer Dieter Maurer
Lev Maximov
Daniel May Daniel May
Madison May Madison May
Lucas Maystre Lucas Maystre

View File

@ -22,6 +22,9 @@ Core and Builtins
Library Library
------- -------
- Issue #26844: Fix error message for imp.find_module() to refer to 'path'
instead of 'name'. Patch by Lev Maximov.
- Issue #23804: Fix SSL zero-length recv() calls to not block and not raise - Issue #23804: Fix SSL zero-length recv() calls to not block and not raise
an error about unclean EOF. an error about unclean EOF.