Project

General

Profile

« Previous | Next » 

Revision 5069c5f5

Added by jeremyevans (Jeremy Evans) almost 6 years ago

Honor refinements for modules that prepend other modules

This previously did not work, and the reason it did not work is
that:

  1. Refining a module or class that prepends other modules places
    the refinements in the class itself and not the origin iclass.

  2. Inclusion of a module that prepends other modules skips the
    module itself, including only iclasses for the prepended modules
    and the origin iclass.

Those two behaviors combined meant that the method table for the
refined methods for the included module never ends up in the
method lookup chain for the class including the module.

Fix this by not skipping the module itself when the module is
included. This requires some code rearranging in
rb_include_class_new to make sure the correct method tables and
origin settings are used for the created iclass.

As origin iclasses shouldn't be exposed to Ruby, this also
requires skipping modules that have origin iclasses in
Module#ancestors (classes that have origin iclasses were already
skipped).

Fixes [Bug #16242]