Skip to content

Commit

Permalink
Don't resolve shadowed arguments variables from functions (#14036)
Browse files Browse the repository at this point in the history
  • Loading branch information
The-x-Theorist committed Dec 13, 2021
1 parent dfcfbf3 commit add64e8
Show file tree
Hide file tree
Showing 6 changed files with 51 additions and 1 deletion.
@@ -1,7 +1,7 @@
var _arguments2 = 1;

function fn() {
var _arguments = _arguments2;
var _arguments = arguments;

var foo = function () {
return _arguments;
Expand Down
@@ -0,0 +1,11 @@
var arguments = [1, 2, 3];
var arr = () => arguments[0];

expect(arr()).toStrictEqual(1)

function foo(n) {
var f = () => arguments[0] + n; // foo's implicit arguments binding. arguments[0] is n
return f();
}

expect(foo(3)).toStrictEqual(6)
@@ -0,0 +1,10 @@
var arguments = [1, 2, 3];
var arr = (n) => arguments[0];

arr(4); // 1

function foo(n) {
var f = () => arguments[0] + n; // foo's implicit arguments binding. arguments[0] is n
return f();
}

@@ -0,0 +1,4 @@
{
"sourceType": "script",
"plugins": ["transform-arrow-functions"]
}
@@ -0,0 +1,18 @@
var _arguments = [1, 2, 3];

var arr = function (n) {
return _arguments[0];
};

arr(4); // 1

function foo(n) {
var _arguments2 = arguments;

var f = function () {
return _arguments2[0] + n;
}; // foo's implicit arguments binding. arguments[0] is n


return f();
}
7 changes: 7 additions & 0 deletions packages/babel-traverse/src/scope/index.ts
Expand Up @@ -1138,6 +1138,13 @@ export default class Scope {
} else {
return binding;
}
} else if (
!binding &&
name === "arguments" &&
scope.path.isFunction() &&
!scope.path.isArrowFunctionExpression()
) {
break;
}
previousPath = scope.path;
} while ((scope = scope.parent));
Expand Down

0 comments on commit add64e8

Please sign in to comment.