[[FIX]] Correct location for error

jshint · Mar 1, 2021 · e831188 · e831188
1 parent 916c230
commit e831188
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/jshint.js b/src/jshint.js
@@ -4772,7 +4772,7 @@ var JSHINT = (function() {
         var tokens = destructuringPattern(context);
         _.each(tokens, function(token) {
           if (token.id) {
-            state.funct["(scope)"].addParam(token.id, token, "exception");
+            state.funct["(scope)"].addParam(token.id, token.token, "exception");
           }
         });
       } else if (state.tokens.next.type !== "(identifier)") {

diff --git a/tests/unit/core.js b/tests/unit/core.js
@@ -1784,6 +1784,18 @@ exports.testDefaultArguments = function (test) {
   test.done();
 };
 
+exports.testEarlyCatchParam = function (test) {
+  TestRun(test)
+    .addError(2, 18, "'y' was used before it was declared, which is illegal for 'exception' variables.")
+    .test([
+      "try {",
+      "} catch ([x = y, y]) {",
+      "}"
+    ], { esversion: 6 });
+
+  test.done();
+};
+
 exports.testDuplicateParamNames = function (test) {
   var src = [
   "(function() {",