Code Review: LexicalScopes2


#1

tfpt review “/shelveset:LexicalScopes2;REDMOND\tomat”

The actual shelveset :slight_smile:

Refactors and simplifies LexicalScope and allocation of local
variables. Previously a LocalVariable instance was created per each
local variable access, which is unnecessary. Since many scopes don’t
define any variables at all we allocate a variable dictionary lazily.
We also used to fallback to dynamic variable lookup in evals in some
cases even though we could lookup statically.

Tomas


#2

Looks good.