From 5d19bb8d5d087c6e7a383974024c67e11b6334a1 Mon Sep 17 00:00:00 2001 From: "Alex Lam S.L" Date: Sat, 12 Dec 2020 21:01:38 +0000 Subject: [PATCH] fix corner case in `booleans` (#4375) fixes #4374 --- lib/ast.js | 15 +++++++-------- test/compress/booleans.js | 28 ++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+), 8 deletions(-) diff --git a/lib/ast.js b/lib/ast.js index 262372ee..31281c1e 100644 --- a/lib/ast.js +++ b/lib/ast.js @@ -1451,14 +1451,13 @@ TreeWalker.prototype = { || 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)); - if (fn.name) return false; - self = this.parent(++i); - if (!self || self.TYPE != "Call" || self.expression !== fn) return false; + for (var call, fn = p; call = this.parent(++i); fn = call) { + if (call.TYPE == "Call") { + if (!(fn instanceof AST_Lambda) || fn.name) return false; + } else if (fn instanceof AST_Lambda) { + return false; + } + } } else { return false; } diff --git a/test/compress/booleans.js b/test/compress/booleans.js index 6e7988bf..779f8d8c 100644 --- a/test/compress/booleans.js +++ b/test/compress/booleans.js @@ -153,3 +153,31 @@ issue_3690: { } expect_stdout: "PASS" } + +issue_4374: { + options = { + booleans: true, + conditionals: true, + if_return: true, + reduce_vars: true, + unused: true, + } + input: { + (function() { + console.log(f()); + function f(a) { + if (null) return 0; + if (a) return 1; + return 0; + } + })(); + } + expect: { + (function() { + console.log(function(a) { + return !null && a ? 1 : 0; + }()); + })(); + } + expect_stdout: "0" +} -- 2.34.1