From: Alex Lam S.L Date: Fri, 25 Dec 2020 12:10:58 +0000 (+0000) Subject: fix corner case in `unused` (#4457) X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=a1b2735dd8f7f89b7dd57ee7098f1720df28838f;p=UglifyJS.git fix corner case in `unused` (#4457) fixes #4456 --- diff --git a/lib/compress.js b/lib/compress.js index 0718e5c7..1dc01f93 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -5568,13 +5568,11 @@ merge(Compressor.prototype, { if (def.value) def.value = def.value.transform(tt); if (def.name instanceof AST_Destructured) { var name = trim_destructured(def.name, def.value, function(node) { - if (node instanceof AST_SymbolDeclaration) { - if (!drop_vars) return node; - if (node.definition().id in in_use_ids) return node; - if (is_catch(node)) return node; - if (is_var && !can_drop_symbol(node)) return node; - return null; - } + if (!drop_vars) return node; + if (node.definition().id in in_use_ids) return node; + if (is_catch(node)) return node; + if (is_var && !can_drop_symbol(node)) return node; + return null; }); if (name) { flush(); @@ -5747,7 +5745,7 @@ merge(Compressor.prototype, { descend(node, tt); if (node.left instanceof AST_Destructured) { var lhs = trim_destructured(node.left, node.right, function(node) { - if (node instanceof AST_SymbolRef) return node; + return node; }); if (!lhs) return node.right; node.left = lhs; diff --git a/test/compress/destructured.js b/test/compress/destructured.js index 7fe2f0c8..98e84f48 100644 --- a/test/compress/destructured.js +++ b/test/compress/destructured.js @@ -2164,3 +2164,36 @@ issue_4446: { expect_stdout: "PASS" node_version: ">=6" } + +issue_4456: { + options = { + pure_getters: true, + unused: true, + } + input: { + var o = { + set p(v) { + console.log(v); + }, + }; + [ function() { + try { + return o; + } catch ({}) {} + }().p ] = [ "PASS" ]; + } + expect: { + var o = { + set p(v) { + console.log(v); + }, + }; + [ function() { + try { + return o; + } catch ({}) {} + }().p ] = [ "PASS" ]; + } + expect_stdout: "PASS" + node_version: ">=6" +}