remove flow types from being scope tracked - these aren't used anywhere and if they're going to be added back they need to be abstracted from bindings to create any real value - fixes #2105

This commit is contained in:
Sebastian McKenzie 2015-07-28 19:26:52 +01:00
parent 73d6257377
commit 2c3566de03

View File

@ -542,8 +542,6 @@ export default class Scope {
this.registerBinding("let", path);
} else if (path.isImportDeclaration() || path.isExportDeclaration()) {
this.registerBinding("module", path);
} else if (path.isFlowDeclaration()) {
this.registerBinding("type", path);
} else {
this.registerBinding("unknown", path);
}
@ -590,9 +588,6 @@ export default class Scope {
for (var id of (ids[name]: Array)) {
var local = this.getOwnBinding(name);
if (local) {
// don't ever let a type alias shadow a local binding
if (kind === "type") continue;
// same identifier so continue safely as we're likely trying to register it
// multiple times
if (local.identifier === id) continue;