[Python-checkins] cpython: Turn _return_module() into _handle_fromlist().

[brett.cannon]

http://hg.python.org/cpython/rev/3670df23081d
changeset:   75235:3670df23081d
parent:      75233:e394e62473ee
user:        Brett Cannon <brett at python.org>
date:        Thu Feb 23 20:47:57 2012 -0500
summary:
  Turn _return_module() into _handle_fromlist().

files:
  Lib/importlib/_bootstrap.py |  50 ++++++++++++------------
  1 files changed, 25 insertions(+), 25 deletions(-)


diff --git a/Lib/importlib/_bootstrap.py b/Lib/importlib/_bootstrap.py
--- a/Lib/importlib/_bootstrap.py
+++ b/Lib/importlib/_bootstrap.py
@@ -1001,7 +1001,7 @@
         return _find_and_load(name, _gcd_import)
 
 
-def _return_module(module, name, fromlist, level, import_):
+def _handle_fromlist(module, fromlist, import_):
     """Figure out what __import__ should return.
 
     The import_ parameter is a callable which takes the name of module to
@@ -1010,29 +1010,18 @@
 
     """
     # The hell that is fromlist ...
-    if not fromlist:
-        # Return up to the first dot in 'name'. This is complicated by the fact
-        # that 'name' may be relative.
-        if level == 0:
-            return sys.modules[name.partition('.')[0]]
-        elif not name:
-            return module
-        else:
-            cut_off = len(name) - len(name.partition('.')[0])
-            return sys.modules[module.__name__[:-cut_off]]
-    else:
-        # If a package was imported, try to import stuff from fromlist.
-        if hasattr(module, '__path__'):
-            if '*' in fromlist and hasattr(module, '__all__'):
-                fromlist = list(fromlist)
-                fromlist.remove('*')
-                fromlist.extend(module.__all__)
-            for x in (y for y in fromlist if not hasattr(module,y)):
-                try:
-                    import_('{0}.{1}'.format(module.__name__, x))
-                except ImportError:
-                    pass
-        return module
+    # If a package was imported, try to import stuff from fromlist.
+    if hasattr(module, '__path__'):
+        if '*' in fromlist and hasattr(module, '__all__'):
+            fromlist = list(fromlist)
+            fromlist.remove('*')
+            fromlist.extend(module.__all__)
+        for x in (y for y in fromlist if not hasattr(module,y)):
+            try:
+                import_('{0}.{1}'.format(module.__name__, x))
+            except ImportError:
+                pass
+    return module
 
 
 def _calc___package__(globals):
@@ -1066,7 +1055,18 @@
     else:
         package = _calc___package__(globals)
         module = _gcd_import(name, package, level)
-    return _return_module(module, name, fromlist, level, _gcd_import)
+    if not fromlist:
+        # Return up to the first dot in 'name'. This is complicated by the fact
+        # that 'name' may be relative.
+        if level == 0:
+            return sys.modules[name.partition('.')[0]]
+        elif not name:
+            return module
+        else:
+            cut_off = len(name) - len(name.partition('.')[0])
+            return sys.modules[module.__name__[:-cut_off]]
+    else:
+        return _handle_fromlist(module, fromlist, _gcd_import)
 
 
 def _setup(sys_module, imp_module):

-- 
Repository URL: http://hg.python.org/cpython