Fixes

> A pointer bug which caused the child tree to not propagate up through the tree
nayas360 · Mar 28, 2020 · 3546df2 · 3546df2
1 parent bd2b827
commit 3546df2
Showing 1 changed file with 8 additions and 13 deletions.
diff --git a/examples/algebrica/ast/expression/operator.go b/examples/algebrica/ast/expression/operator.go
@@ -10,27 +10,22 @@ type Operator struct {
 	E node.Expression
 }
 
-func (o Operator) Parse(tokens *goply.TokenStream) error {
+func (o *Operator) Parse(tokens *goply.TokenStream) error {
 	if tokens.EOS() {
 		return ast.EndOfTokenStreamErr
 	}
-	var op node.Operator
 	if o.E != nil {
-		op = &Binary{Left: o.E}
+		o.E = &Binary{Left: o.E}
 	} else {
-		op = &Binary{}
+		// no left context unary operator
+		o.E = &Binary{}
 	}
-	err := op.Parse(tokens)
-	if err != nil {
-		return err
-	}
-	o.E = op
-	return nil
+	return o.E.Parse(tokens)
 }
 
-func (o Operator) String() string {
+func (o *Operator) String() string {
 	return o.E.String()
 }
 
-func (o Operator) Expression() {}
-func (o Operator) Operator()   {}
+func (o *Operator) Expression() {}
+func (o *Operator) Operator()   {}