Commits

Benjamin Peterson committed fdd70fa

switch assertion to an explicit ValueError

Comments (0)

Files changed (2)

Lib/lib2to3/fixer_util.py

     """Find the top level namespace."""
     # Scamper up to the top level namespace
     while node.type != syms.file_input:
-        assert node.parent, "Tree is insane! root found before "\
-                           "file_input node was found."
         node = node.parent
+        if not node:
+            raise ValueError("root found before file_input node was found.")
     return node
 
 def does_tree_import(package, name, node):

Lib/lib2to3/refactor.py

 
                         try:
                             find_root(node)
-                        except AssertionError:
+                        except ValueError:
                             # this node has been cut off from a
                             # previous transformation ; skip
                             continue