Code Review: LexicalScopes2

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.


Looks good.

This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs