Forum: IronRuby Code Review: VisibilityFixes1

Announcement (2017-05-07): is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see and for other Rails- und Ruby-related community platforms.
Tomas M. (Guest)
on 2009-06-03 01:59
(Received via mailing list)
Attachment: VisibilityFixes1.diff (0 Bytes)
tfpt review "/shelveset:VisibilityFixes1;REDMOND\tomat"

  Fixes bugs:
public(:foo) does not work correctly for mixed-in methods
visibility of send :define_method Title is required

  Factors super-forwarders into a separate SuperForwarderInfo <:
RubyMemberInfo (it used to be just a flag on member info). The
super-forwarder needs to remember the name of the method to forward to
since that can change if it's aliased (see Visibility2C test case).

Curt H. (Guest)
on 2009-06-03 02:23
(Received via mailing list)
Looks good. One small quibble: the comment in RubyModule.ForEachMember
"visit the member even if it doesn't have the right visibility so that
any overridden member with the right visibility won't later be visited"
isn't quite right as the member isn't actually being visited; it's just
marked that way in the dict. :)
This topic is locked and can not be replied to.