diff options
| -rw-r--r-- | lib/compress.js | 12 | ||||
| -rw-r--r-- | test/compress/arrays.js | 69 |
2 files changed, 81 insertions, 0 deletions
diff --git a/lib/compress.js b/lib/compress.js index 7577437e..0a89de7e 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -374,12 +374,20 @@ merge(Compressor.prototype, { var lhs = is_lhs(node, parent); if (lhs) return lhs; if (parent instanceof AST_Array) return is_modified(compressor, tw, parent, parent, level + 1); + if (parent instanceof AST_Binary) { + if (!lazy_op[parent.operator]) return; + return is_modified(compressor, tw, parent, parent, level + 1); + } if (parent instanceof AST_Call) { return !immutable && parent.expression === node && !parent.is_expr_pure(compressor) && (!(value instanceof AST_LambdaExpression) || !(parent instanceof AST_New) && value.contains_this()); } + if (parent instanceof AST_Conditional) { + if (parent.condition === node) return; + return is_modified(compressor, tw, parent, parent, level + 1); + } if (parent instanceof AST_ForEnumeration) return parent.init === node; if (parent instanceof AST_ObjectKeyVal) { if (parent.value !== node) return; @@ -391,6 +399,10 @@ merge(Compressor.prototype, { var prop = read_property(value, parent); return (!immutable || recursive) && is_modified(compressor, tw, parent, prop, level + 1); } + if (parent instanceof AST_Sequence) { + if (parent.tail_node() !== node) return; + return is_modified(compressor, tw, parent, value, level + 1, immutable, recursive); + } } function is_arguments(def) { diff --git a/test/compress/arrays.js b/test/compress/arrays.js index fff7e9b6..4767a9a5 100644 --- a/test/compress/arrays.js +++ b/test/compress/arrays.js @@ -355,3 +355,72 @@ constructor_good: { expect_stdout: true expect_warnings: [] } + +unsafe_evaluate_modified_binary: { + options = { + evaluate: true, + reduce_vars: true, + unsafe: true, + } + input: { + (function(a) { + (console && a).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect: { + (function(a) { + (console && a).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect_stdout: "PASS" +} + +unsafe_evaluate_modified_conditional: { + options = { + evaluate: true, + reduce_vars: true, + unsafe: true, + } + input: { + (function(a) { + (console ? a : []).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect: { + (function(a) { + (console ? a : []).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect_stdout: "PASS" +} + +unsafe_evaluate_modified_sequence: { + options = { + evaluate: true, + reduce_vars: true, + unsafe: true, + } + input: { + (function(a) { + (0, a).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect: { + (function(a) { + (0, a).push(1); + if (a.length) + console.log("PASS"); + })([]); + } + expect_stdout: "PASS" +} |