cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Handle case where optimisation concist of only replacing rest param id with args

Browse Source
This commit is contained in:
Victor Felder 2015-12-18 17:43:00 +01:00
parent 6cc0538d2f
commit 5d414a2323
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
packages/babel-plugin-transform-es2015-parameters/src/rest.js
View File

@ -69,6 +69,7 @@ let memberExpressionOptimisationVisitor = {
if (parentPath.isMemberExpression({ computed: false, object: node })) {
let prop = parentPath.get("property");
if (prop.node.name === "length") {
state.replaceOnly = true;
state.candidates.push(path);
return;
}
@ -144,7 +145,10 @@ export let visitor = {
name: rest.name,
// whether any references to the rest parameter were made in a function
deopted: false
deopted: false,
// whether all we need to do is replace rest parameter identifier with 'arguments'
replaceOnly: false
};
path.traverse(memberExpressionOptimisationVisitor, state);
@ -154,7 +158,9 @@ export let visitor = {
if (state.candidates.length) {
for (let candidate of (state.candidates: Array)) {
candidate.replaceWith(argsId);
optimiseCandidate(candidate.parent, candidate.parentPath, state.offset);
if (!state.replaceOnly) {
optimiseCandidate(candidate.parent, candidate.parentPath, state.offset);
}
}
}
return;