:update: remove useless adopt_children since the bug was fixed in grako

link/utils/__init__.py

@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
 
-__version__ = '1.6'
+__version__ = '2.0'
 
 CONF_BASE_PATH = 'link/utils'

link/utils/grammar.py

@@ -47,36 +47,6 @@ def codegenerator(modname, prefix, grammar):
     return module
 
 
-def adopt_children(ast, parent=None):
-    """
-    Make sure that grako Nodes have the parent property set correctly.
-
-    :param ast: Grako AST
-    :type ast: grako.model.Node, grako.util.Mapping or list
-
-    :param parent: Parent Node
-    :type parent: grako.model.Node or None
-    """
-
-    childset = set()
-
-    if isinstance(ast, Node) and ast not in childset:
-        if isinstance(parent, Node):
-            ast._parent = parent
-            childset.add(ast)
-
-    elif isinstance(ast, Mapping):
-        for c in ast.values():
-            adopt_children(c, parent=parent)
-
-    elif isinstance(ast, list):
-        for c in ast:
-            adopt_children(c, parent=parent)
-
-    for child in childset:
-        adopt_children(child._ast, parent=child)
-
-
 def find_ancestor(node, classname):
     """
     Find first node's ancestor which match class' name.

link/utils/test/grammar.py

@@ -3,7 +3,7 @@
 from b3j0f.utils.ut import UTCase
 from unittest import main
 
-from link.utils.grammar import codegenerator, adopt_children, find_ancestor
+from link.utils.grammar import codegenerator, find_ancestor
 from grako.model import ModelBuilderSemantics, Node
 from grako.exceptions import FailedToken
 import sys
@@ -34,7 +34,6 @@ def test_adopt_children(self):
         model = parser.parse('DSL', rule_name='dsl')
         self.assertIsInstance(model, Node)
 
-        adopt_children(model._ast, parent=model)
         self.assertIs(model.sub[0].parent, model)
 
     def test_find_ancestor(self):
@@ -46,7 +45,6 @@ def test_find_ancestor(self):
 
         parser = mod.MyDSLParser(semantics=ModelBuilderSemantics())
         model = parser.parse('DSL', rule_name='dsl')
-        adopt_children(model._ast, parent=model)
         pnode = find_ancestor(model.sub.sub, 'RootNode')
 
         self.assertIs(pnode, model)