diff --git a/lib/compress.js b/lib/compress.js index 1eb6bbff..56b9e8ce 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -749,15 +749,23 @@ merge(Compressor.prototype, { var candidates = []; var stat_index = statements.length; while (--stat_index >= 0) { + // Treat parameters as collapsible in IIFE, i.e. + // function(a, b){ ... }(x()); + // would be translated into equivalent assignments: + // var a = x(), b = undefined; + if (stat_index == 0 && compressor.option("unused")) extract_args(); + // Find collapsible assignments extract_candidates(statements[stat_index]); while (candidates.length > 0) { var candidate = candidates.pop(); var lhs = get_lhs(candidate); if (!lhs || is_lhs_read_only(lhs)) continue; + // Locate symbols which may execute code outside of scanning range var lvalues = get_lvalues(candidate); if (lhs instanceof AST_SymbolRef) lvalues[lhs.name] = false; var side_effects = value_has_side_effects(candidate); - var hit = false, abort = false, replaced = false; + var hit = candidate.name instanceof AST_SymbolFunarg; + var abort = false, replaced = false; var tt = new TreeTransformer(function(node, descend) { if (abort) return node; // Skip nodes before `candidate` as quickly as possible @@ -840,6 +848,39 @@ merge(Compressor.prototype, { } } + function extract_args() { + var iife, fn = compressor.self(); + if (is_func_expr(fn) + && !fn.name + && !fn.uses_arguments + && !fn.uses_eval + && (iife = compressor.parent()) instanceof AST_Call + && iife.expression === fn + && all(iife.args, function(arg) { + return !(arg instanceof AST_Expansion); + })) { + fn.argnames.forEach(function(sym, i) { + if (!(sym instanceof AST_SymbolFunarg)) return; + var arg = iife.args[i]; + if (!arg) arg = make_node(AST_Undefined, sym); + else arg.walk(new TreeWalker(function(node) { + if (!arg) return true; + if (node instanceof AST_SymbolRef && fn.variables.has(node.name)) { + var s = node.definition().scope; + if (s !== scope) while (s = s.parent_scope) { + if (s === scope) return true; + } + arg = null; + } + })); + if (arg) candidates.push(make_node(AST_VarDef, sym, { + name: sym, + value: arg + })); + }); + } + } + function extract_candidates(expr) { if (expr instanceof AST_Assign && !expr.left.has_side_effects(compressor) || expr instanceof AST_Unary && (expr.operator == "++" || expr.operator == "--")) { @@ -860,7 +901,7 @@ merge(Compressor.prototype, { function get_lhs(expr) { if (expr instanceof AST_VarDef && expr.name instanceof AST_SymbolDeclaration) { var def = expr.name.definition(); - if (def.orig.length > 1 + if (def.orig.length > 1 && !(expr.name instanceof AST_SymbolFunarg) || def.references.length == 1 && !compressor.exposed(def)) { return make_node(AST_SymbolRef, expr.name, expr.name); } @@ -903,6 +944,14 @@ merge(Compressor.prototype, { } function remove_candidate(expr) { + if (expr.name instanceof AST_SymbolFunarg) { + var index = compressor.self().argnames.indexOf(expr.name); + var args = compressor.parent().args; + if (args[index]) args[index] = make_node(AST_Number, args[index], { + value: 0 + }); + return true; + } var found = false; return statements[stat_index].transform(new TreeTransformer(function(node, descend, in_list) { if (found) return node; @@ -3343,98 +3392,54 @@ merge(Compressor.prototype, { var value = stat.value; if (!value || value.is_constant_expression()) { var args = self.args.concat(value || make_node(AST_Undefined, self)); - return make_sequence(self, args).transform(compressor); + return make_sequence(self, args).optimize(compressor); } } if (is_func_expr(exp) && !exp.is_generator && !exp.async) { if (compressor.option("inline") + && simple_args && !exp.name - && (exp.body instanceof AST_Node || exp.body.length == 1) && !exp.uses_arguments && !exp.uses_eval - && simple_args + && (exp.body instanceof AST_Node || exp.body.length == 1) + && all(exp.argnames, function(arg) { + return arg.__unused; + }) && !self.has_pure_annotation(compressor)) { var value; if (stat instanceof AST_Return) { - value = stat.value.clone(true); + value = stat.value; } else if (stat instanceof AST_SimpleStatement) { value = make_node(AST_UnaryPrefix, stat, { operator: "void", - expression: stat.body.clone(true) + expression: stat.body }); } if (value) { - var fn = exp.clone(); - fn.argnames = []; - fn.body = []; - if (exp.argnames.length > 0) { - fn.body.push(make_node(AST_Var, self, { - definitions: exp.argnames.map(function(sym, i) { - if (sym instanceof AST_Expansion) { - return make_node(AST_VarDef, sym, { - name: sym.expression, - value: make_node(AST_Array, self, { - elements: self.args.slice(i).map(function(arg) { - return arg.clone(true); - }) - }) - }); - } - var arg = self.args[i]; - return make_node(AST_VarDef, sym, { - name: sym, - value: arg ? arg.clone(true) : make_node(AST_Undefined, self) - }); - }) - })); - } - if (self.args.length > exp.argnames.length && !(exp.argnames[exp.argnames.length - 1] instanceof AST_Expansion)) { - fn.body.push(make_node(AST_SimpleStatement, self, { - body: make_sequence(self, self.args.slice(exp.argnames.length).map(function(node) { - return node.clone(true); - })) - })); - } - fn.body.push(make_node(AST_Return, self, { - value: value - })); - var body = fn.transform(compressor).body; - if (body instanceof AST_Node) { - body = [ - make_node(AST_Return, body, { - value: body - }) - ]; - } - if (body.length == 0) return make_node(AST_Undefined, self); - if (body.length == 1 && body[0] instanceof AST_Return) { - value = body[0].value; - if (!value) return make_node(AST_Undefined, self); - var tw = new TreeWalker(function(node) { - if (value === self) return true; - if (node instanceof AST_SymbolRef) { - var ref = node.scope.find_variable(node); - if (ref && ref.scope.parent_scope === fn.parent_scope) { - value = self; - return true; - } - } - if (node instanceof AST_This && !tw.find_parent(AST_Scope)) { - value = self; + var tw = new TreeWalker(function(node) { + if (!value) return true; + if (node instanceof AST_SymbolRef) { + var ref = node.scope.find_variable(node); + if (ref && ref.scope.parent_scope === fn.parent_scope) { + value = null; return true; } - }); - value.walk(tw); - if (value !== self) value = best_of(compressor, value, self); - } else { - value = self; - } - if (value !== self) return value; + } + if (node instanceof AST_This && !tw.find_parent(AST_Scope)) { + value = null; + return true; + } + }); + value.walk(tw); + } + if (value) { + var args = self.args.concat(value); + return make_sequence(self, args).optimize(compressor); } } if (compressor.option("side_effects") && !(exp.body instanceof AST_Node) && all(exp.body, is_empty)) { var args = self.args.concat(make_node(AST_Undefined, self)); - return make_sequence(self, args).transform(compressor); + return make_sequence(self, args).optimize(compressor); } } if (compressor.option("drop_console")) { diff --git a/package.json b/package.json index 5249184d..738dcac5 100644 --- a/package.json +++ b/package.json @@ -4,7 +4,7 @@ "homepage": "https://github.com/mishoo/UglifyJS2/tree/harmony", "author": "Mihai Bazon (http://lisperator.net/)", "license": "BSD-2-Clause", - "version": "3.0.22", + "version": "3.0.23", "engines": { "node": ">=0.8.0" }, diff --git a/test/compress/arrow.js b/test/compress/arrow.js index 99376da2..2d9eb73b 100644 --- a/test/compress/arrow.js +++ b/test/compress/arrow.js @@ -287,6 +287,7 @@ issue_2105_1: { collapse_vars: true, ecma: 6, inline: true, + passes: 3, reduce_vars: true, side_effects: true, unused: true, @@ -312,7 +313,7 @@ issue_2105_1: { }); } expect: { - void (() => { + (() => { var quux = () => { console.log("PASS"); }; @@ -332,6 +333,7 @@ issue_2105_2: { options = { collapse_vars: true, inline: true, + passes: 2, reduce_vars: true, side_effects: true, unused: true, @@ -357,7 +359,7 @@ issue_2105_2: { }); } expect: { - void (() => { + (() => { var quux = () => { console.log("PASS"); }; @@ -372,7 +374,7 @@ issue_2105_2: { expect_stdout: "PASS" node_version: ">=6" } - +/* issue_2136_2: { options = { arrows: true, @@ -428,7 +430,7 @@ issue_2136_3: { expect_stdout: "2" node_version: ">=6" } - +*/ call_args: { options = { arrows: true, diff --git a/test/compress/collapse_vars.js b/test/compress/collapse_vars.js index 302da2b9..aa9a2e41 100644 --- a/test/compress/collapse_vars.js +++ b/test/compress/collapse_vars.js @@ -2077,10 +2077,10 @@ chained_3: { } expect: { console.log(function(a, b) { - var c = a, c = b; + var c = 1, c = b; b++; return c; - }(1, 2)); + }(0, 2)); } expect_stdout: "2" } @@ -2330,3 +2330,73 @@ reassign_const_2: { } expect_stdout: true } + +issue_2187_1: { + options = { + collapse_vars: true, + unused: true, + } + input: { + var a = 1; + !function(foo) { + foo(); + var a = 2; + console.log(a); + }(function() { + console.log(a); + }); + } + expect: { + var a = 1; + !function(foo) { + foo(); + var a = 2; + console.log(a); + }(function() { + console.log(a); + }); + } + expect_stdout: [ + "1", + "2", + ] +} + +issue_2187_2: { + options = { + collapse_vars: true, + unused: true, + } + input: { + var b = 1; + console.log(function(a) { + return a && ++b; + }(b--)); + } + expect: { + var b = 1; + console.log(function(a) { + return b-- && ++b; + }()); + } + expect_stdout: "1" +} + +issue_2187_3: { + options = { + collapse_vars: true, + inline: true, + unused: true, + } + input: { + var b = 1; + console.log(function(a) { + return a && ++b; + }(b--)); + } + expect: { + var b = 1; + console.log(b-- && ++b); + } + expect_stdout: "1" +} diff --git a/test/compress/drop-unused.js b/test/compress/drop-unused.js index 0a810470..18170dec 100644 --- a/test/compress/drop-unused.js +++ b/test/compress/drop-unused.js @@ -1299,6 +1299,7 @@ issue_2105: { options = { collapse_vars: true, inline: true, + passes: 3, reduce_vars: true, side_effects: true, unused: true, @@ -1324,7 +1325,7 @@ issue_2105: { }); } expect: { - !void function() { + (function() { var quux = function() { console.log("PASS"); }; @@ -1334,7 +1335,7 @@ issue_2105: { quux(); } }; - }().prop(); + })().prop(); } expect_stdout: "PASS" } @@ -1357,7 +1358,7 @@ issue_2136_1: { expect_stdout: "[]" node_version: ">=6" } - +/* issue_2136_2: { options = { collapse_vars: true, @@ -1409,7 +1410,7 @@ issue_2136_3: { expect_stdout: "2" node_version: ">=6" } - +*/ issue_2163: { options = { pure_funcs: [ "pure" ], diff --git a/test/compress/functions.js b/test/compress/functions.js index 677c36e8..36495b7a 100644 --- a/test/compress/functions.js +++ b/test/compress/functions.js @@ -468,11 +468,9 @@ issue_2114_1: { } expect: { var c = 0; - !function() { - 0; - }((c += 1, c = 1 + c, function() { + c = 1 + (c += 1), function() { var b = void (b && (b.b += (c += 1, 0))); - }())); + }(); console.log(c); } expect_stdout: "2" diff --git a/test/compress/issue-281.js b/test/compress/issue-281.js index 9b8c8bfd..65871a84 100644 --- a/test/compress/issue-281.js +++ b/test/compress/issue-281.js @@ -419,7 +419,7 @@ wrap_iife_in_return_call: { expect_exact: '(void console.log("test"))();' } -pure_annotation: { +pure_annotation_1: { options = { inline: true, side_effects: true, @@ -432,6 +432,20 @@ pure_annotation: { expect_exact: "" } +pure_annotation_2: { + options = { + collapse_vars: true, + inline: true, + side_effects: true, + } + input: { + /*@__PURE__*/(function(n) { + console.log("hello", n); + }(42)); + } + expect_exact: "" +} + drop_fargs: { options = { cascade: true, @@ -449,9 +463,7 @@ drop_fargs: { } expect: { var a = 1; - !function() { - a++; - }(++a && a.var); + ++a && a.var, a++; console.log(a); } expect_stdout: "3" @@ -474,9 +486,7 @@ keep_fargs: { } expect: { var a = 1; - !function(a_1) { - a++; - }(++a && a.var); + ++a && a.var, a++; console.log(a); } expect_stdout: "3" diff --git a/test/mocha/glob.js b/test/mocha/glob.js index 8567ecb3..eb5d477d 100644 --- a/test/mocha/glob.js +++ b/test/mocha/glob.js @@ -26,12 +26,12 @@ describe("bin/uglifyjs with input file globs", function() { }); }); it("bin/uglifyjs with multiple input file globs.", function(done) { - var command = uglifyjscmd + ' "test/input/issue-1242/???.es5" "test/input/issue-1242/*.js" -mc toplevel'; + var command = uglifyjscmd + ' "test/input/issue-1242/???.es5" "test/input/issue-1242/*.js" -mc toplevel,passes=2'; exec(command, function(err, stdout) { if (err) throw err; - assert.strictEqual(stdout, 'var print=console.log.bind(console);print("qux",9,6),print("Foo:",2*11);\n'); + assert.strictEqual(stdout, 'var print=console.log.bind(console);print("qux",9,6),print("Foo:",22);\n'); done(); }); });