Fix extra parens for update expressions (#796)

* Remove parens around UpdateExpression when parent is CallExpression. * Add new test case.
prettier · Feb 23, 2017 · c8c1875 · c8c1875
1 parent 7c4654b
commit c8c1875
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/fast-path.js b/src/fast-path.js
@@ -258,7 +258,6 @@ FPp.needsParens = function(assumeExpressionContext) {
           return name === "object" && parent.object === node;
 
         case "TaggedTemplateExpression":
-        case "CallExpression":
         case "NewExpression":
           return true;
 

diff --git a/tests/update_expression/__snapshots__/jsfmt.spec.js.snap b/tests/update_expression/__snapshots__/jsfmt.spec.js.snap
@@ -4,9 +4,11 @@ exports[`update_expression.js 1`] = `
 "(this.x++).toString()
 new (r++);
 (x++)();
+const uuid = String(this._uuidCounter++);
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 (this.x++).toString();
 new (r++)();
 (x++)();
+const uuid = String(this._uuidCounter++);
 "
 `;
diff --git a/tests/update_expression/update_expression.js b/tests/update_expression/update_expression.js
@@ -1,3 +1,4 @@
 (this.x++).toString()
 new (r++);
 (x++)();
+const uuid = String(this._uuidCounter++);