]> granicus.if.org Git - python/commitdiff
Merged revisions 82542 via svnmerge from
authorBenjamin Peterson <benjamin@python.org>
Sun, 4 Jul 2010 16:49:46 +0000 (16:49 +0000)
committerBenjamin Peterson <benjamin@python.org>
Sun, 4 Jul 2010 16:49:46 +0000 (16:49 +0000)
svn+ssh://pythondev@svn.python.org/python/branches/py3k

................
  r82542 | benjamin.peterson | 2010-07-04 11:44:15 -0500 (Sun, 04 Jul 2010) | 17 lines

  Merged revisions 81478,82530-82531 via svnmerge from
  svn+ssh://pythondev@svn.python.org/sandbox/trunk/2to3/lib2to3

  ........
    r81478 | benjamin.peterson | 2010-05-22 13:47:39 -0500 (Sat, 22 May 2010) | 1 line

    ensure doctests have some future_features
  ........
    r82530 | benjamin.peterson | 2010-07-04 11:11:41 -0500 (Sun, 04 Jul 2010) | 1 line

    simplify ignore star imports from itertools #8892
  ........
    r82531 | benjamin.peterson | 2010-07-04 11:13:20 -0500 (Sun, 04 Jul 2010) | 1 line

    wrap with parenthesis not \
  ........
................

Lib/lib2to3/fixes/fix_itertools_imports.py
Lib/lib2to3/tests/test_fixers.py

index 1897e3647f893862b0dc0857faac78fcdebb1721..f4b4ead5feafd154770f7713853c073b12a55741 100644 (file)
@@ -20,6 +20,9 @@ class FixItertoolsImports(fixer_base.BaseFix):
             if child.type == token.NAME:
                 member = child.value
                 name_node = child
+            elif child.type == token.STAR:
+                # Just leave the import as is.
+                return
             else:
                 assert child.type == syms.import_as_name
                 name_node = child.children[0]
@@ -44,8 +47,8 @@ class FixItertoolsImports(fixer_base.BaseFix):
             children[-1].remove()
 
         # If there are no imports left, just get rid of the entire statement
-        if not (imports.children or getattr(imports, 'value', None)) or \
-                imports.parent is None:
+        if (not (imports.children or getattr(imports, 'value', None)) or
+            imports.parent is None):
             p = node.prefix
             node = BlankLine()
             node.prefix = p
index 855b4bb71f6a670ac4ddbe2ea49ba7d9da75ca76..91853a4a1277e7bbe65da3fe66fd90164449651b 100644 (file)
@@ -3670,6 +3670,10 @@ class Test_itertools_imports(FixerTestCase):
         a = "from itertools import bar, filterfalse, foo"
         self.check(b, a)
 
+    def test_import_star(self):
+        s = "from itertools import *"
+        self.unchanged(s)
+
 
     def test_unchanged(self):
         s = "from itertools import foo"