account for declaration assignment in collapse_vars (#2859)
fixes #2858
This commit is contained in:
@@ -990,6 +990,7 @@ merge(Compressor.prototype, {
|
|||||||
|| node instanceof AST_Debugger
|
|| node instanceof AST_Debugger
|
||||||
|| node instanceof AST_IterationStatement && !(node instanceof AST_For)
|
|| node instanceof AST_IterationStatement && !(node instanceof AST_For)
|
||||||
|| node instanceof AST_Try
|
|| node instanceof AST_Try
|
||||||
|
|| node instanceof AST_VarDef && node.value && side_effects && !references_in_scope(node.name.definition())
|
||||||
|| node instanceof AST_With
|
|| node instanceof AST_With
|
||||||
|| parent instanceof AST_For && node !== parent.init
|
|| parent instanceof AST_For && node !== parent.init
|
||||||
|| (side_effects || !replace_all)
|
|| (side_effects || !replace_all)
|
||||||
|
|||||||
@@ -4190,3 +4190,37 @@ return_3: {
|
|||||||
}
|
}
|
||||||
expect_stdout: "0"
|
expect_stdout: "0"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
issue_2858: {
|
||||||
|
options = {
|
||||||
|
collapse_vars: true,
|
||||||
|
unused: true,
|
||||||
|
}
|
||||||
|
input: {
|
||||||
|
var b;
|
||||||
|
(function() {
|
||||||
|
function f() {
|
||||||
|
a++;
|
||||||
|
}
|
||||||
|
f();
|
||||||
|
var c = f();
|
||||||
|
var a = void 0;
|
||||||
|
c || (b = a);
|
||||||
|
})();
|
||||||
|
console.log(b);
|
||||||
|
}
|
||||||
|
expect: {
|
||||||
|
var b;
|
||||||
|
(function() {
|
||||||
|
function f() {
|
||||||
|
a++;
|
||||||
|
}
|
||||||
|
f();
|
||||||
|
var c = f();
|
||||||
|
var a = void 0;
|
||||||
|
c || (b = a);
|
||||||
|
})();
|
||||||
|
console.log(b);
|
||||||
|
}
|
||||||
|
expect_stdout: "undefined"
|
||||||
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user