From 8fcc7c18b35591afe73fa0472894918f20dbb998 Mon Sep 17 00:00:00 2001 From: Lucas Duailibe Date: Mon, 17 Jun 2019 06:14:16 -0300 Subject: [PATCH] Printing arguments: short circuiting some cases (#6229) --- src/language-js/printer-estree.js | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/src/language-js/printer-estree.js b/src/language-js/printer-estree.js index 3de462223..2d471db38 100644 --- a/src/language-js/printer-estree.js +++ b/src/language-js/printer-estree.js @@ -4128,20 +4128,15 @@ function printArgumentsList(path, options, print) { const somePrintedArgumentsWillBreak = printedArguments.some(willBreak); + const simpleConcat = concat(["(", concat(printedExpanded), ")"]); + return concat([ somePrintedArgumentsWillBreak ? breakParent : "", conditionalGroup( [ - concat([ - ifBreak( - indent(concat(["(", softline, concat(printedExpanded)])), - concat(["(", concat(printedExpanded)]) - ), - somePrintedArgumentsWillBreak - ? concat([ifBreak(maybeTrailingComma), softline]) - : "", - ")" - ]), + !somePrintedArgumentsWillBreak + ? simpleConcat + : ifBreak(allArgsBrokenOut(), simpleConcat), shouldGroupFirst ? concat([ "(",