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

Safely transform multiple rest arguments

Browse Source
This commit is contained in:
Victor Felder
2015-12-14 13:49:47 +01:00
parent 3dfb4eb8c6
commit 2dea8b2352
3 changed files with 19 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -121,7 +121,7 @@ export let visitor = {
argsId._shadowedFunctionLiteral = path;
function optimiseCandidate(parent, parentPath, offset) {
if (t.isReturnStatement(parentPath.parent) || t.isIdentifier(parentPath.parent.id)) {
if (t.isReturnStatement(parentPath.parent) || t.isIdentifier(parentPath.parent.id) || t.isIdentifier(parentPath.parent.left)) {
parentPath.replaceWith(loadRest({
ARGUMENTS: argsId,
INDEX: t.numericLiteral(parent.property.value + offset)

7
packages/babel-plugin-transform-es2015-parameters/test/fixtures/parameters/rest-multiple/actual.js vendored
View File

@@ -9,3 +9,10 @@ function t(f, ...items) {
x = items[0];
x = items[1];
}
function u(f, g, ...items) {
var x = f;
var y = g;
x = items[0];
y = items[1];
}

15
packages/babel-plugin-transform-es2015-parameters/test/fixtures/parameters/rest-multiple/expected.js vendored
View File

@@ -1,11 +1,18 @@
var t = function (f) {
var x = f;
x = arguments[1];
x = arguments[2];
x = arguments.length <= 1 ? undefined : arguments[1];
x = arguments.length <= 2 ? undefined : arguments[2];
};
function t(f) {
var x = f;
x = arguments[1];
x = arguments[2];
x = arguments.length <= 1 ? undefined : arguments[1];
x = arguments.length <= 2 ? undefined : arguments[2];
}
function u(f, g) {
var x = f;
var y = g;
x = arguments.length <= 2 ? undefined : arguments[2];
y = arguments.length <= 3 ? undefined : arguments[3];
}