Forum: IronRuby Code Review: LexicalScopes2

Announcement (2017-05-07): www.ruby-forum.com is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see rubyonrails.org/community and ruby-lang.org/en/community for other Rails- und Ruby-related community platforms.
Tomas M. (Guest)
on 2008-10-30 05:39
(Received via mailing list)
Attachment: LexicalScopes2.diff (0 Bytes)
tfpt review "/shelveset:LexicalScopes2;REDMOND\tomat"

The actual shelveset :)

  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
Curt H. (Guest)
on 2008-10-30 18:52
(Received via mailing list)
Looks good.
This topic is locked and can not be replied to.