exercise 1 completed

1.b now works
1.c implened error for "function takes N positional arguments but M were given"

1 files changed, 13 insertions, 5 deletions

diff --git a/src/ast/nodes/ParamdefNode.java b/src/ast/nodes/ParamdefNode.java
index 265b6b6..5164537 100644
--- a/src/ast/nodes/ParamdefNode.java
+++ b/src/ast/nodes/ParamdefNode.java
@@ -1,24 +1,32 @@
 package ast.nodes;
 
-import java.util.ArrayList;
-
 import ast.types.*;
-import semanticanalysis.*;
+import java.util.ArrayList;
+import semanticanalysis.SemanticError;
+import semanticanalysis.SymbolTable;
 
 /**
  * Node for the `paramdef` statement of the grammar. Extends the `AtomNode`
  * class.
  */
 public class ParamdefNode extends AtomNode {
+
     public ParamdefNode(String val) {
         super(val);
     }
 
     @Override
     public ArrayList<SemanticError> checkSemantics(SymbolTable ST, int _nesting) {
-        ST.insert(this.getId(), this.typeCheck(), _nesting, "");
+        var errors = new ArrayList<SemanticError>();
+        String paramName = this.getId();
+
+        if (!ST.top_lookup(paramName)) {
+            ST.insert(paramName, this.typeCheck(), _nesting, "");
+        } else {
+            errors.add(new SemanticError("Duplicate argument '" + paramName + "' in function definition."));
+        }
 
-        return new ArrayList<SemanticError>();
+        return errors;
     }
 
     // FIXME: it should returns the param' type