disallow collapse_vars constant replacement in for-in statements (#1543)

2017-03-03 13:39:54 -05:00
parent 07accd2fbb
commit ce54c9ccee
1 changed files with 3 additions and 5 deletions
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -525,11 +525,9 @@ merge(Compressor.prototype, {
                    // Constant single use vars can be replaced in any scope.
                    if (var_decl.value.is_constant()) {
                        var ctt = new TreeTransformer(function(node) {
-                            if (node === ref) {
-                                var parent = ctt.parent();
-                                if (!(parent instanceof AST_ForIn && parent.init === node)) {
-                                    return replace_var(node, parent, true);
-                                }
+                            if (node === ref
+                                && !ctt.find_parent(AST_ForIn)) {
+                                return replace_var(node, ctt.parent(), true);
                            }
                        });
                        stat.transform(ctt);