Forum: IronRuby Code Review: NoStaticSites4

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.
Cb51033949ffccd982ae32c9f890f25a?d=identicon&s=25 Tomas Matousek (Guest)
on 2008-11-26 18:30
(Received via mailing list)
Attachment: NoStaticSites4.diff (600 KB)
tfpt review "/shelveset:NoStaticSites4;REDMOND\tomat"

Adds support for SiteLocalStorage to IronRuby binder. This allows us to
store arbitrary data in dynamic site rules including other dynamic
sites. Using this feature library methods that dynamically invoke other
Ruby methods should now take as parameters all sites they're using
instead of calling to statically allocated sites. This will allows us to
remove runtime checks from the rules and also decreases the degree of
polymorphism for the sites (since static sites are used from many places
with variety of targets they tend to be megamorphic).

This is first part of the change. Many static sites still remain to be
removed. Some library methods (e.g. Range#step) are highly dynamic. Some
method need refactoring to reduce the number of sites.

Tomas
F983f0c990cba2fe743ef62a975ec99c?d=identicon&s=25 Curt Hagenlocher (Guest)
on 2008-11-26 21:51
(Received via mailing list)
Looks good.
This topic is locked and can not be replied to.