vitalif-js
/
prettier
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix cases of missing parens with NewExpression (#230)

master
Brian Ng 2017-01-15 22:56:40 -06:00 committed by James Long
parent 0e1afd6312
commit 340a39b414
4 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/fast-path.js
View File

@ -239,6 +239,7 @@ FPp.needsParens = function(assumeExpressionContext) {
case "LogicalExpression":
switch (parent.type) {
case "CallExpression":
case "NewExpression":
return name === "callee" && parent.callee === node;
case "UnaryExpression":
@ -340,6 +341,7 @@ FPp.needsParens = function(assumeExpressionContext) {
case "SpreadProperty":
case "BinaryExpression":
case "LogicalExpression":
case "NewExpression":
return true;
case "CallExpression":

8
tests/new_expression/__snapshots__/jsfmt.spec.js.snap Normal file
View File

@ -0,0 +1,8 @@
exports[`test new_expression.js 1`] = `
"new (memoize.Cache || MapCache)
new (typeof this == \"function\" ? this : Dict())
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
new (memoize.Cache || MapCache)();
new (typeof this == \"function\" ? this : Dict())();
"
`;

1
tests/new_expression/jsfmt.spec.js Normal file
View File

@ -0,0 +1 @@
run_spec(__dirname);

2
tests/new_expression/new_expression.js Normal file
View File

@ -0,0 +1,2 @@
new (memoize.Cache || MapCache)
new (typeof this == "function" ? this : Dict())