Skip to content

Commit 4886636

Browse files
committed
Refactoring and optimize Parse.js
1 parent 5fde639 commit 4886636

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

lib/Parser.js

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -844,9 +844,9 @@ Parser.prototype.walkClassExpression = function walkClassExpression(expression)
844844
Parser.prototype.walkCallExpression = function walkCallExpression(expression) {
845845
var result;
846846

847-
function walkIIFE(functionExpression, args) {
847+
function walkIIFE(functionExpression, options) {
848848
var params = functionExpression.params;
849-
args = args.map(function(arg) {
849+
var args = options.map(function(arg) {
850850
var renameIdentifier = this.getRenameIdentifier(arg);
851851
if(renameIdentifier && this.applyPluginsBailResult1("can-rename " + renameIdentifier, arg)) {
852852
if(!this.applyPluginsBailResult1("rename " + renameIdentifier, arg))
@@ -857,11 +857,12 @@ Parser.prototype.walkCallExpression = function walkCallExpression(expression) {
857857
this.inScope(params.filter(function(identifier, idx) {
858858
return !args[idx];
859859
}), function() {
860-
args.forEach(function(arg, idx) {
861-
if(!arg) return;
862-
if(!params[idx] || params[idx].type !== "Identifier") return;
863-
this.scope.renames["$" + params[idx].name] = arg;
864-
}, this);
860+
for(var i = 0, len = args.length; i < len; i++) {
861+
var param = args[i];
862+
if(!param) continue;
863+
if(!params[i] || params[i].type !== "Identifier") continue;
864+
this.scope.renames["$" + params[i].name] = param;
865+
}
865866
if(functionExpression.body.type === "BlockStatement")
866867
this.walkStatement(functionExpression.body);
867868
else

0 commit comments

Comments
 (0)