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

Merge pull request #3411 from babel/hotfix-7197

Browse Source 
[hotfix T7197] Use scope.moveBindingTo
This commit is contained in:
Amjad Masad 2016-03-09 14:00:01 -08:00
commit facf6bc82f
4 changed files with 22 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
packages/babel-plugin-transform-es2015-block-scoping/src/index.js
View File

@ -90,9 +90,10 @@ function convertBlockScopedToVar(path, parent, scope, moveBindingsToParent = fal
const parentScope = scope.getFunctionParent();
const ids = path.getBindingIdentifiers();
for (let name in ids) {
scope.removeOwnBinding(name);
let binding = scope.getOwnBinding(name);
if (binding) binding.kind = "var";
scope.moveBindingTo(name, parentScope);
}
parentScope.registerBinding("var", path);
}
}
@ -351,8 +352,8 @@ class BlockScoping {
const binding = scope.getBinding(ref.name);
if (!binding) continue;
if (binding.kind === "let" || binding.kind === "const") {
scope.removeOwnBinding(ref.name);
parentScope.registerBinding("var", binding.path);
binding.kind = "var";
scope.moveBindingTo(ref.name, parentScope);
}
}
}

5
packages/babel-plugin-transform-es2015-block-scoping/test/fixtures/general/function-name/actual.js vendored Normal file
View File

@ -0,0 +1,5 @@
let foo = () => {
foo = () => { };
};
foo();

5
packages/babel-plugin-transform-es2015-block-scoping/test/fixtures/general/function-name/expected.js vendored Normal file
View File

@ -0,0 +1,5 @@
var _foo = function foo() {
_foo = function foo() {};
};
_foo();

7
packages/babel-plugin-transform-es2015-block-scoping/test/fixtures/general/function-name/options.json vendored Normal file
View File

@ -0,0 +1,7 @@
{
"plugins": [
"transform-es2015-arrow-functions",
"transform-es2015-function-name",
"transform-es2015-block-scoping"
]
}