diff options
author | Alex Lam S.L <alexlamsl@gmail.com> | 2019-10-10 09:37:02 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-10-10 09:37:02 +0800 |
commit | 33c94d3bd90af2ceaa3faf36cdf19c2dd8700619 (patch) | |
tree | bcffafb1f07c09ec229b7fe48e0e009f80b4907f | |
parent | b18f717b46d655aa183d09109e1e6cbbc4462c04 (diff) | |
download | tracifyjs-33c94d3bd90af2ceaa3faf36cdf19c2dd8700619.tar.gz tracifyjs-33c94d3bd90af2ceaa3faf36cdf19c2dd8700619.zip |
detect boolean context across IIFEs (#3461)
-rw-r--r-- | lib/ast.js | 8 | ||||
-rw-r--r-- | test/compress/evaluate.js | 31 |
2 files changed, 39 insertions, 0 deletions
@@ -979,6 +979,14 @@ TreeWalker.prototype = { || p instanceof AST_Conditional || p.tail_node() === self) { self = p; + } else if (p instanceof AST_Return) { + var fn; + do { + fn = this.parent(++i); + if (!fn) return false; + } while (!(fn instanceof AST_Lambda)); + self = this.parent(++i); + if (!self || self.TYPE != "Call" || self.expression !== fn) return false; } else { return false; } diff --git a/test/compress/evaluate.js b/test/compress/evaluate.js index 0ac117fe..22c87b70 100644 --- a/test/compress/evaluate.js +++ b/test/compress/evaluate.js @@ -1757,3 +1757,34 @@ issue_3387_2: { } expect_stdout: "NaN" } + +iife_boolean_context: { + options = { + booleans: true, + evaluate: true, + } + input: { + console.log(function() { + return Object(1) || false; + }() ? "PASS" : "FAIL"); + console.log(function() { + return [].length || true; + }() ? "PASS" : "FAIL"); + } + expect: { + console.log(function() { + return Object(1); + }() ? "PASS" : "FAIL"); + console.log(function() { + return [].length, 1; + }() ? "PASS" : "FAIL"); + } + expect_stdout: [ + "PASS", + "PASS", + ] + expect_warnings: [ + "WARN: Dropping side-effect-free || [test/compress/evaluate.js:2,19]", + "WARN: Boolean || always true [test/compress/evaluate.js:5,19]", + ] +} |