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

Merge pull request #3161 from loganfsmyth/function-rename-6781

Browse Source 
Only rename outer function bindings on conflict - fixes T6781
This commit is contained in:
Logan Smyth 2015-12-13 23:44:43 -08:00
commit 7b25b33ac6
3 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/babel-plugin-transform-es2015-function-name/test/fixtures/function-name/function-collision/actual.js vendored
View File

@ -16,4 +16,8 @@ function f() {
b;
}
};
function commit(b) {
b();
}
});

4
packages/babel-plugin-transform-es2015-function-name/test/fixtures/function-name/function-collision/expected.js vendored
View File

@ -16,4 +16,8 @@ function _f() {
_b;
}
};
function commit(b) {
b();
}
});

2
packages/babel-traverse/src/scope/lib/renamer.js
View File

@ -15,7 +15,7 @@ let renameVisitor = {
},
"AssignmentExpression|Declaration"(path, state) {
let ids = path.getBindingIdentifiers();
let ids = path.getOuterBindingIdentifiers();
for (let name in ids) {
if (name === state.oldName) ids[name].name = state.newName;