diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2006-11-07 08:56:18 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2006-11-07 08:56:18 +0000 |
commit | 6f1c934bc361ec5d01a0b4b1a45d07840af02dc3 (patch) | |
tree | 6fc07fb23fb397b3ce755307112d8253d0c1fc52 /class.c | |
parent | 568602aeeaa1636a8f74dcf967e15ebd0e843da7 (diff) | |
download | ruby-6f1c934bc361ec5d01a0b4b1a45d07840af02dc3.tar.gz |
* class.c (rb_include_module): revert duplicate inclusion of
modules. [ruby-dev:29793]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@11291 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'class.c')
-rw-r--r-- | class.c | 24 |
1 files changed, 21 insertions, 3 deletions
@@ -344,7 +344,7 @@ include_class_new(VALUE module, VALUE super) void rb_include_module(VALUE klass, VALUE module) { - VALUE c; + VALUE p, c; int changed = 0; rb_frozen_class_p(klass); @@ -359,11 +359,29 @@ rb_include_module(VALUE klass, VALUE module) OBJ_INFECT(klass, module); c = klass; while (module) { + int superclass_seen = Qfalse; + if (RCLASS(klass)->m_tbl == RCLASS(module)->m_tbl) rb_raise(rb_eArgError, "cyclic include detected"); - RCLASS(c)->super = include_class_new(module, RCLASS(c)->super); - c = RCLASS(c)->super; + /* ignore if the module included already in superclasses */ + for (p = RCLASS(klass)->super; p; p = RCLASS(p)->super) { + switch (BUILTIN_TYPE(p)) { + case T_ICLASS: + if (RCLASS(p)->m_tbl == RCLASS(module)->m_tbl) { + if (!superclass_seen) { + c = p; /* move insertion point */ + } + goto skip; + } + break; + case T_CLASS: + superclass_seen = Qtrue; + break; + } + } + c = RCLASS(c)->super = include_class_new(module, RCLASS(c)->super); changed = 1; + skip: module = RCLASS(module)->super; } if (changed) rb_clear_cache(); |