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

deopt on binding identifiers with same rest name - fixes #2091

Browse Source
This commit is contained in:
Sebastian McKenzie 2015-07-28 01:19:59 +01:00
parent 507303456f
commit 57d2323ae9
3 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
packages/babel/src/transformation/transformers/es6/parameters/rest.js
View File

@ -78,6 +78,18 @@ var memberExpressionOptimisationVisitor = {
state.references.push(this);
}
},
/**
* Deopt on use of a binding identifier with the same name as our rest param.
*
* See https://github.com/babel/babel/issues/2091
*/
BindingIdentifier(node, parent, scope, state) {
if (node.name === state.name) {
state.deopted = true;
}
}
};

1
packages/babel/test/fixtures/transformation/es6.parameters/rest-binding-deoptimisation/actal.js vendored Normal file
View File

@ -0,0 +1 @@
const deepAssign = (...args) => args = [];

9
packages/babel/test/fixtures/transformation/es6.parameters/rest-binding-deoptimisation/expected.js vendored Normal file
View File

@ -0,0 +1,9 @@
"use strict";
var deepAssign = function deepAssign() {
for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
args[_key] = arguments[_key];
}
return args = [];
};