Rename Tree.appendChild() to Tree.append_child()

Author: boriel

src/ast/tree.py

@@ -71,12 +71,12 @@ def postorder(self):
 
         yield self
 
-    def appendChild(self, node: 'Tree'):
+    def append_child(self, node: 'Tree'):
         """ Appends the given node to the current children list
         """
         self.children.append(node)
 
-    def prependChild(self, node: 'Tree'):
+    def prepend_child(self, node: 'Tree'):
         """ Inserts the given node at the beginning of the children list
         """
         self.children.insert(0, node)

src/symbols/arglist.py

@@ -24,7 +24,7 @@ def args(self):
     def args(self, value):
         for i in value:
             assert isinstance(value, SymbolARGUMENT)
-            self.appendChild(i)
+            self.append_child(i)
 
     def __getitem__(self, range_):
         return self.args[range_]
@@ -56,6 +56,6 @@ def make_node(cls, node, *args):
 
         for arg in args:
             assert isinstance(arg, SymbolARGUMENT)
-            node.appendChild(arg)
+            node.append_child(arg)
 
         return node

src/symbols/boundlist.py

@@ -9,6 +9,8 @@
 #                    the GNU General License
 # ----------------------------------------------------------------------
 
+from typing import Optional
+
 from .symbol_ import Symbol
 from .bound import SymbolBOUND
 
@@ -32,7 +34,7 @@ def __str__(self):
         return '(%s)' % ', '.join(str(x) for x in self)
 
     @classmethod
-    def make_node(cls, node, *args):
+    def make_node(cls, node: Optional[Symbol], *args):
         """ Creates an array BOUND LIST.
         """
         if node is None:
@@ -44,6 +46,6 @@ def make_node(cls, node, *args):
         for arg in args:
             if arg is None:
                 continue
-            node.appendChild(arg)
+            node.append_child(arg)
 
         return node

src/symbols/paramlist.py

@@ -44,14 +44,14 @@ def make_node(cls, node, *params):
 
         for i in params:
             if i is not None:
-                node.appendChild(i)
+                node.append_child(i)
 
         return node
 
-    def appendChild(self, param):
+    def append_child(self, param):
         """ Overrides base class.
         """
-        Symbol.appendChild(self, param)
+        Symbol.append_child(self, param)
         if param.offset is None:
             param.offset = self.size
             self.size += param.size

src/symbols/symbol_.py

@@ -26,7 +26,7 @@ def __init__(self, *children):
         self._t = None
         for child in children:
             assert isinstance(child, Symbol)
-            self.appendChild(child)
+            self.append_child(child)
 
         self._required_by: Counter = Counter()  # Symbols that depends on this one
         self._requires: Counter = Counter()  # Symbols this one depends on

src/zxbasm/asmparse.py

@@ -322,7 +322,7 @@ def makenode(cls, symbol, *nexts):
                 continue
             if not isinstance(i, cls):
                 raise NotAnAstError(i)
-            result.appendChild(i)
+            result.append_child(i)
 
         return result
 

src/zxbc/zxbparser.py

@@ -403,7 +403,7 @@ def make_call(id_: str, lineno: int, args: symbols.ARGLIST):
             offset = make_typecast(TYPE.uinteger,
                                    make_number(arr.offset, lineno=lineno),
                                    lineno)
-            arr.appendChild(offset)
+            arr.append_child(offset)
         return arr
 
     if entry.class_ == CLASS.var:  # An already declared/used string var
@@ -532,7 +532,7 @@ def p_start(p):
 
     if not is_null(ast):
         if isinstance(ast, symbols.BLOCK) and not is_ender(ast[-1]):
-            ast.appendChild(__end)
+            ast.append_child(__end)
     else:
         ast = __end
 
@@ -554,11 +554,11 @@ def p_start(p):
 
     # Appends variable declarations at the end.
     for var in SYMBOL_TABLE.vars_:
-        data_ast.appendChild(make_var_declaration(var))
+        data_ast.append_child(make_var_declaration(var))
 
     # Appends arrays declarations at the end.
     for var in SYMBOL_TABLE.arrays:
-        data_ast.appendChild(make_array_declaration(var))
+        data_ast.append_child(make_array_declaration(var))
 
 
 def p_program_program_line(p):
@@ -1514,7 +1514,7 @@ def p_if_then_part(p):
 def p_if_inline(p):
     """ statement : if_inline else_part_inline
     """
-    p[1].appendChild(make_block(p[2][0], p[2][1]))
+    p[1].append_child(make_block(p[2][0], p[2][1]))
     p[0] = p[1]
 
 
@@ -1544,7 +1544,7 @@ def p_for_sentence(p):
     p[0] = p[1]
     if is_null(p[0]):
         return
-    p[1].appendChild(make_block(p[2], p[3]))
+    p[1].append_child(make_block(p[2], p[3]))
     gl.LOOPS.pop()
 
 
@@ -2020,18 +2020,18 @@ def p_print_list(p):
     p[0].eol = (p[3] is not None)
 
     if p[3] is not None:
-        p[0].appendChild(p[3])
+        p[0].append_child(p[3])
 
 
 def p_print_list_comma(p):
     """ print_list : print_list COMMA print_elem
     """
     p[0] = p[1]
     p[0].eol = (p[3] is not None)
-    p[0].appendChild(make_sentence(p.lineno(2), 'PRINT_COMMA'))
+    p[0].append_child(make_sentence(p.lineno(2), 'PRINT_COMMA'))
 
     if p[3] is not None:
-        p[0].appendChild(p[3])
+        p[0].append_child(p[3])
 
 
 def p_print_list_at(p):
@@ -2943,7 +2943,7 @@ def p_function_header_pre(p):
     forwarded = p[1].entry.forwarded
 
     p[0] = p[1]
-    p[0].appendChild(p[2])
+    p[0].append_child(p[2])
     p[0].params_size = p[2].size
     lineno = p.lineno(3)