enhance collapse_vars (#2788)
This commit is contained in:
@@ -1181,14 +1181,26 @@ merge(Compressor.prototype, {
|
||||
expr.definitions.forEach(extract_candidates);
|
||||
} else if (expr instanceof AST_DWLoop) {
|
||||
extract_candidates(expr.condition);
|
||||
if (!(expr.body instanceof AST_Block)) {
|
||||
extract_candidates(expr.body);
|
||||
}
|
||||
} else if (expr instanceof AST_Exit) {
|
||||
if (expr.value) extract_candidates(expr.value);
|
||||
} else if (expr instanceof AST_For) {
|
||||
if (expr.init) extract_candidates(expr.init);
|
||||
if (expr.condition) extract_candidates(expr.condition);
|
||||
if (expr.step) extract_candidates(expr.step);
|
||||
if (!(expr.body instanceof AST_Block)) {
|
||||
extract_candidates(expr.body);
|
||||
}
|
||||
} else if (expr instanceof AST_If) {
|
||||
extract_candidates(expr.condition);
|
||||
if (!(expr.body instanceof AST_Block)) {
|
||||
extract_candidates(expr.body);
|
||||
}
|
||||
if (expr.alternative && !(expr.alternative instanceof AST_Block)) {
|
||||
extract_candidates(expr.alternative);
|
||||
}
|
||||
} else if (expr instanceof AST_Sequence) {
|
||||
expr.expressions.forEach(extract_candidates);
|
||||
} else if (expr instanceof AST_SimpleStatement) {
|
||||
@@ -1215,10 +1227,12 @@ merge(Compressor.prototype, {
|
||||
if (parent instanceof AST_Call) return node;
|
||||
if (parent instanceof AST_Case) return node;
|
||||
if (parent instanceof AST_Conditional) return node;
|
||||
if (parent instanceof AST_Definitions) return find_stop(parent, level + 1);
|
||||
if (parent instanceof AST_Exit) return node;
|
||||
if (parent instanceof AST_If) return node;
|
||||
if (parent instanceof AST_IterationStatement) return node;
|
||||
if (parent instanceof AST_Sequence) return find_stop(parent, level + 1);
|
||||
if (parent instanceof AST_SimpleStatement) return find_stop(parent, level + 1);
|
||||
if (parent instanceof AST_Switch) return node;
|
||||
if (parent instanceof AST_VarDef) return node;
|
||||
return null;
|
||||
|
||||
@@ -4012,3 +4012,53 @@ replace_all_var: {
|
||||
}
|
||||
expect_stdout: "PASS"
|
||||
}
|
||||
|
||||
cascade_statement: {
|
||||
options = {
|
||||
collapse_vars: true,
|
||||
}
|
||||
input: {
|
||||
function f1(a, b) {
|
||||
var c;
|
||||
if (a)
|
||||
return c = b, c || a;
|
||||
else
|
||||
c = a, c(b);
|
||||
}
|
||||
function f2(a, b) {
|
||||
var c;
|
||||
while (a)
|
||||
c = b, a = c + b;
|
||||
do
|
||||
throw c = a + b, c;
|
||||
while (c);
|
||||
}
|
||||
function f3(a, b) {
|
||||
for (; a < b; a++)
|
||||
if (c = a, c && b)
|
||||
var c = (c = b(a), c);
|
||||
}
|
||||
}
|
||||
expect: {
|
||||
function f1(a, b) {
|
||||
var c;
|
||||
if (a)
|
||||
return (c = b) || a;
|
||||
else
|
||||
(c = a)(b);
|
||||
}
|
||||
function f2(a, b) {
|
||||
var c;
|
||||
while (a)
|
||||
a = (c = b) + b;
|
||||
do
|
||||
throw c = a + b;
|
||||
while (c);
|
||||
}
|
||||
function f3(a, b) {
|
||||
for (; a < b; a++)
|
||||
if ((c = a) && b)
|
||||
var c = c = b(a);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user