[[FIX]] Allow parentheses around object destructuring assignment.

Fixes gh-2775
jshint · Feb 14, 2016 · 7a0bd70 · 7a0bd70
1 parent 97d0ac1
commit 7a0bd70
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/src/jshint.js b/src/jshint.js
@@ -2575,7 +2575,9 @@ var JSHINT = (function() {
           // Used to delineate an integer number literal from a dereferencing
           // punctuator (otherwise interpreted as a decimal point)
           (ret.type === "(number)" &&
-            checkPunctuator(pn, ".") && /^\d+$/.test(ret.value));
+            checkPunctuator(pn, ".") && /^\d+$/.test(ret.value)) ||
+          // Used to wrap object destructuring assignment
+          (opening.beginsStmt && ret.id === "=" && ret.left.id === "{");
       }
     }
 

diff --git a/tests/unit/options.js b/tests/unit/options.js
@@ -3047,6 +3047,26 @@ singleGroups.postfix = function (test) {
   test.done();
 };
 
+singleGroups.destructuringAssign = function (test) {
+
+  var code = [
+    // statements
+    "({ x } = { x : 1 });",
+    "([ x ] = [ 1 ]);",
+    // expressions
+    "1, ({ x } = { x : 1 });",
+    "1, ([ x ] = [ 1 ]);"
+  ];
+
+  TestRun(test)
+    .addError(2, "Unnecessary grouping operator.")
+    .addError(3, "Unnecessary grouping operator.")
+    .addError(4, "Unnecessary grouping operator.")
+    .test(code, { esversion: 6, singleGroups: true, expr: true });
+
+  test.done();
+};
+
 exports.elision = function (test) {
   var code = [
     "var a = [1,,2];",