aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/compress.js34
-rw-r--r--test/compress/drop-unused.js44
2 files changed, 61 insertions, 17 deletions
diff --git a/lib/compress.js b/lib/compress.js
index f5989341..e14e63ae 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -2944,18 +2944,18 @@ merge(Compressor.prototype, {
OPT(AST_Call, function(self, compressor){
var exp = self.expression;
- if (compressor.option("reduce_vars") && exp instanceof AST_SymbolRef) {
- var fixed = exp.fixed_value();
- if (fixed instanceof AST_Function) exp = fixed;
- }
+ var fn = exp;
if (compressor.option("unused")
- && exp instanceof AST_Function
- && !exp.uses_arguments
- && !exp.uses_eval) {
+ && (fn instanceof AST_Function
+ || compressor.option("reduce_vars")
+ && fn instanceof AST_SymbolRef
+ && (fn = fn.fixed_value()) instanceof AST_Function)
+ && !fn.uses_arguments
+ && !fn.uses_eval) {
var pos = 0, last = 0;
for (var i = 0, len = self.args.length; i < len; i++) {
- var trim = i >= exp.argnames.length;
- if (trim || exp.argnames[i].__unused) {
+ var trim = i >= fn.argnames.length;
+ if (trim || fn.argnames[i].__unused) {
var node = self.args[i].drop_side_effect_free(compressor);
if (node) {
self.args[pos++] = node;
@@ -3156,15 +3156,15 @@ merge(Compressor.prototype, {
}
}
}
- if (exp instanceof AST_Function) {
- var stat = exp.body[0];
- if (compressor.option("inline") && stat instanceof AST_Return) {
- var value = stat && 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);
- }
+ var stat = fn instanceof AST_Function && fn.body[0];
+ if (compressor.option("inline") && stat instanceof AST_Return) {
+ 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);
}
+ }
+ if (exp instanceof AST_Function) {
if (compressor.option("inline")
&& !exp.name
&& exp.body.length == 1
diff --git a/test/compress/drop-unused.js b/test/compress/drop-unused.js
index af792bfa..08fc7b18 100644
--- a/test/compress/drop-unused.js
+++ b/test/compress/drop-unused.js
@@ -1108,3 +1108,47 @@ var_catch_toplevel: {
}();
}
}
+
+issue_2105: {
+ options = {
+ collapse_vars: true,
+ inline: true,
+ reduce_vars: true,
+ side_effects: true,
+ unused: true,
+ }
+ input: {
+ !function(factory) {
+ factory();
+ }( function() {
+ return function(fn) {
+ fn()().prop();
+ }( function() {
+ function bar() {
+ var quux = function() {
+ console.log("PASS");
+ }, foo = function() {
+ console.log;
+ quux();
+ };
+ return { prop: foo };
+ }
+ return bar;
+ } );
+ });
+ }
+ expect: {
+ !void function() {
+ var quux = function() {
+ console.log("PASS");
+ };
+ return {
+ prop: function() {
+ console.log;
+ quux();
+ }
+ };
+ }().prop();
+ }
+ expect_stdout: "PASS"
+}