ClassName declarations are considered block scoped, when they are function scoped. about shift-java HOT 4 CLOSED

I can't tell from your sample what the issue is. However, it's correct that there are two variables created. Test:

class A { static m(){ return A; } }; let foo = A; A = 0; foo.m(); // prints the class, not 0

from shift-java.

@bakkot I see. The issue seems to be that ScopeLookup can only provide one Variable per BindingIdentifier within . We could either merge ClassName and ClassDeclaration, as both function and block scoped, or do a breaking change to allow multiple variables to be returned by findVariableDeclaredBy, or add a new method to allow returning multiple.

from shift-java.

@Protryon:

add a new method to allow returning multiple

I think we already have such a method.

from shift-java.

@bakkot Works for me.

from shift-java.