diff options
| -rw-r--r-- | lib/output.js | 7 | ||||
| -rw-r--r-- | test/compress/functions.js | 10 |
2 files changed, 15 insertions, 2 deletions
diff --git a/lib/output.js b/lib/output.js index bd1a70c2..cf0b41cd 100644 --- a/lib/output.js +++ b/lib/output.js @@ -576,6 +576,7 @@ function OutputStream(options) { indentation : function() { return indentation }, current_width : function() { return current_col - indentation }, should_break : function() { return options.width && this.current_width() >= options.width }, + has_parens : function() { return OUTPUT.slice(-1) == "(" }, newline : newline, print : print, space : space, @@ -683,7 +684,7 @@ function OutputStream(options) { // a function expression needs parens around it when it's provably // the first token to appear in a statement. PARENS(AST_Function, function(output){ - if (first_in_statement(output)) { + if (!output.has_parens() && first_in_statement(output)) { return true; } @@ -704,7 +705,9 @@ function OutputStream(options) { // same goes for an object literal, because otherwise it would be // interpreted as a block of code. - PARENS(AST_Object, first_in_statement); + PARENS(AST_Object, function(output){ + return !output.has_parens() && first_in_statement(output); + }); PARENS(AST_Unary, function(output){ var p = output.parent(); diff --git a/test/compress/functions.js b/test/compress/functions.js index 29c438ae..5d0be0f9 100644 --- a/test/compress/functions.js +++ b/test/compress/functions.js @@ -2012,3 +2012,13 @@ issue_2898: { } expect_stdout: "2" } + +deduplicate_parenthesis: { + input: { + ({}).a = b; + (({}).a = b)(); + (function() {}).a = b; + ((function() {}).a = b)(); + } + expect_exact: "({}).a=b;({}.a=b)();(function(){}).a=b;(function(){}.a=b)();" +} |