Browse Source

DeadCodeInjection: removed renaming of identifiers inside collected block statements

sanex3339 8 years ago
parent
commit
726811c12e

+ 1 - 10
dist/index.js

@@ -5364,16 +5364,7 @@ var DeadCodeInjectionTransformer = DeadCodeInjectionTransformer_1 = function (_A
             if (!Node_1.Node.isBlockStatementNode(targetNode) || !DeadCodeInjectionTransformer_1.isValidBlockStatementNode(targetNode)) {
             if (!Node_1.Node.isBlockStatementNode(targetNode) || !DeadCodeInjectionTransformer_1.isValidBlockStatementNode(targetNode)) {
                 return;
                 return;
             }
             }
-            var clonedBlockStatementNode = NodeUtils_1.NodeUtils.clone(targetNode);
-            estraverse.replace(clonedBlockStatementNode, {
-                enter: function enter(node, parentNode) {
-                    if (Node_1.Node.isIdentifierNode(node)) {
-                        node.name = RandomGeneratorUtils_1.RandomGeneratorUtils.getRandomVariableName(6);
-                    }
-                    return node;
-                }
-            });
-            collectedBlockStatements.push(clonedBlockStatementNode);
+            collectedBlockStatements.push(NodeUtils_1.NodeUtils.clone(targetNode));
         }
         }
     }, {
     }, {
         key: "replaceBlockStatementNodes",
         key: "replaceBlockStatementNodes",

+ 1 - 13
src/node-transformers/dead-code-injection-transformers/DeadCodeInjectionTransformer.ts

@@ -38,19 +38,7 @@ export class DeadCodeInjectionTransformer extends AbstractNodeTransformer {
             return;
             return;
         }
         }
 
 
-        const clonedBlockStatementNode: ESTree.BlockStatement = <ESTree.BlockStatement>NodeUtils.clone(targetNode);
-
-        estraverse.replace(clonedBlockStatementNode, {
-            enter: (node: ESTree.Node, parentNode: ESTree.Node): any => {
-                if (Node.isIdentifierNode(node)) {
-                    node.name = RandomGeneratorUtils.getRandomVariableName(6);
-                }
-
-                return node;
-            }
-        });
-
-        collectedBlockStatements.push(clonedBlockStatementNode);
+        collectedBlockStatements.push(<ESTree.BlockStatement>NodeUtils.clone(targetNode));
     }
     }
 
 
     /**
     /**