JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

reduce AST_ForIn gracefully (#4087)

Browse Source
This commit is contained in:
Alex Lam S.L
2020-09-02 01:51:43 +01:00
committed by GitHub
parent 83a3cbf151
commit 2f0da2ff05
1 changed files with 10 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
test/reduce.js
View File

@@ -112,19 +112,18 @@ module.exports = function reduce_test(testcase, minify_options, reduce_options)
// no structural AST changes before this point. // no structural AST changes before this point.
if (node.start._permute >= REPLACEMENTS.length) return; if (node.start._permute >= REPLACEMENTS.length) return;
if (parent instanceof U.AST_Assign // ignore lvalues
&& parent.left === node if (parent instanceof U.AST_Assign && parent.left === node) return;
|| parent instanceof U.AST_Unary if (parent instanceof U.AST_Unary && parent.expression === node) switch (parent.operator) {
&& parent.expression === node case "++":
&& ["++", "--", "delete"].indexOf(parent.operator) >= 0) { case "--":
// ignore lvalues case "delete":
return; return;
} }
if ((parent instanceof U.AST_For || parent instanceof U.AST_ForIn) // preserve for (var xxx; ...)
&& parent.init === node && node instanceof U.AST_Var) { if (parent instanceof U.AST_For && parent.init === node && node instanceof U.AST_Var) return node;
// preserve for (var ...) // preserve for (xxx in ...)
return node; if (parent instanceof U.AST_ForIn && parent.init === node) return node;
}
// node specific permutations with no parent logic // node specific permutations with no parent logic