Correctly check for duplicate parameter names

Those are not allowed as soon as we have default values for parameters
or rest arguments.

Change-Id: I7dec826c37e6045e4dd1f6b0adb90301efe33daf
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>

1 files changed, 12 insertions, 0 deletions

diff --git a/src/qml/parser/qqmljsast_p.h b/src/qml/parser/qqmljsast_p.h
index 61ee89ad2f..51f3d78588 100644
--- a/src/qml/parser/qqmljsast_p.h
+++ b/src/qml/parser/qqmljsast_p.h
@@ -2099,6 +2099,18 @@ public:
         return n;
     }
 
+    bool isSimpleParameterList()
+    {
+        AST::FormalParameterList *formals = this;
+        while (formals) {
+            if (formals->isRest || formals->defaultExpression)
+                return false;
+            formals = formals->next;
+        }
+        return true;
+    }
+
+
     void accept0(Visitor *visitor) override;
 
     SourceLocation firstSourceLocation() const override