Skip to content
  • Jeremy Evans's avatar
    a0579f36
    Make prepending a refined module after inclusion not break refinements · a0579f36
    Jeremy Evans authored
    After the previous commit, this was still broken. The reason it
    was broken is that a refined module that hasn't been prepended to
    yet keeps the refined methods in the module's method table. When
    prepending, the module's method table is moved to the origin
    iclass, and then the refined methods are moved from the method
    table to a new method table in the module itself.
    
    Unfortunately, that means that if a class has included the module,
    prepending breaks the refinements, because when the methods are
    moved from the origin iclass method table to the module method
    table, they are removed from the method table from the iclass
    created when the module was included earlier.
    
    Fix this by always creating an origin class when including a
    module that has any refinements, even if the refinements are
    not currently used.  I wasn't sure the best way to do that.
    The approach I choose was to use an object flag. The flag is
    set on the module when Module#refine is called, and if the
    flag is present when the module is included in another module
    or class, an origin iclass is created for the module.
    
    Fixes [Bug #13446]
    a0579f36
    Make prepending a refined module after inclusion not break refinements
    Jeremy Evans authored
    After the previous commit, this was still broken. The reason it
    was broken is that a refined module that hasn't been prepended to
    yet keeps the refined methods in the module's method table. When
    prepending, the module's method table is moved to the origin
    iclass, and then the refined methods are moved from the method
    table to a new method table in the module itself.
    
    Unfortunately, that means that if a class has included the module,
    prepending breaks the refinements, because when the methods are
    moved from the origin iclass method table to the module method
    table, they are removed from the method table from the iclass
    created when the module was included earlier.
    
    Fix this by always creating an origin class when including a
    module that has any refinements, even if the refinements are
    not currently used.  I wasn't sure the best way to do that.
    The approach I choose was to use an object flag. The flag is
    set on the module when Module#refine is called, and if the
    flag is present when the module is included in another module
    or class, an origin iclass is created for the module.
    
    Fixes [Bug #13446]
Loading