diff --git a/class.c b/class.c index 24d78a2751..cf0b7b821f 100644 --- a/class.c +++ b/class.c @@ -1145,8 +1145,8 @@ rb_include_module(VALUE klass, VALUE module) iclass = iclass->next; } - int do_include = 1; while (iclass) { + int do_include = 1; VALUE check_class = iclass->klass; /* During lazy sweeping, iclass->klass could be a dead object that * has not yet been swept. */ diff --git a/test/ruby/test_module.rb b/test/ruby/test_module.rb index 6129f0cbbe..24785affa0 100644 --- a/test/ruby/test_module.rb +++ b/test/ruby/test_module.rb @@ -784,6 +784,18 @@ class TestModule < Test::Unit::TestCase assert_equal([:m1, :m0, :m, :sc, :m1, :m0, :c], sc.new.m) end + def test_include_into_module_after_prepend_bug_20871 + bar = Module.new{def bar; 'bar'; end} + foo = Module.new{def foo; 'foo'; end} + m = Module.new + c = Class.new{include m} + m.prepend bar + Class.new{include m} + m.include foo + assert_include c.ancestors, foo + assert_equal "foo", c.new.foo + end + def test_protected_include_into_included_module m1 = Module.new do def other_foo(other) diff --git a/version.h b/version.h index 4a89a8d51e..8e995a483a 100644 --- a/version.h +++ b/version.h @@ -11,7 +11,7 @@ # define RUBY_VERSION_MINOR RUBY_API_VERSION_MINOR #define RUBY_VERSION_TEENY 6 #define RUBY_RELEASE_DATE RUBY_RELEASE_YEAR_STR"-"RUBY_RELEASE_MONTH_STR"-"RUBY_RELEASE_DAY_STR -#define RUBY_PATCHLEVEL 238 +#define RUBY_PATCHLEVEL 239 #include "ruby/version.h" #include "ruby/internal/abi.h"