diff options
author | Alex Lam S.L <alexlamsl@gmail.com> | 2019-10-23 01:04:00 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-10-23 01:04:00 +0800 |
commit | a53ab9937835f795c4714e981d10249e7829d2d4 (patch) | |
tree | a65b27d0c5255643fa0fecdd59a8cba86f1455f1 | |
parent | 02308a7b5651ce04703fafc43887636642c5e888 (diff) | |
download | tracifyjs-a53ab9937835f795c4714e981d10249e7829d2d4.tar.gz tracifyjs-a53ab9937835f795c4714e981d10249e7829d2d4.zip |
fix corner case in `side_effects` (#3514)
fixes #3512
-rw-r--r-- | lib/compress.js | 2 | ||||
-rw-r--r-- | test/compress/functions.js | 30 |
2 files changed, 31 insertions, 1 deletions
diff --git a/lib/compress.js b/lib/compress.js index 2b61e91c..a8bf0d9d 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -4196,7 +4196,7 @@ merge(Compressor.prototype, { var left = this.left; if (left instanceof AST_PropAccess) { var expr = left.expression; - if (expr instanceof AST_Assign && !expr.may_throw_on_access(compressor)) { + if (expr instanceof AST_Assign && expr.operator == "=" && !expr.may_throw_on_access(compressor)) { expr.write_only = "p"; } if (compressor.has_directive("use strict") && expr.is_constant()) return this; diff --git a/test/compress/functions.js b/test/compress/functions.js index 5c8533de..0607e85c 100644 --- a/test/compress/functions.js +++ b/test/compress/functions.js @@ -3340,3 +3340,33 @@ issue_3506_3: { } expect_stdout: "PASS" } + +issue_3512: { + options = { + collapse_vars: true, + pure_getters: "strict", + sequences: true, + side_effects: true, + unused: true, + } + input: { + var a = "PASS"; + (function(b) { + (function() { + b <<= this || 1; + b.a = "FAIL"; + })(); + })(); + console.log(a); + } + expect: { + var a = "PASS"; + (function(b) { + (function() { + (b <<= this || 1).a = "FAIL"; + })(); + })(), + console.log(a); + } + expect_stdout: "PASS" +} |