Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

* class.c (rb_mod_init_copy): Clear the cached inspect string of a

  dup'd anonymous module or class.  [ruby-trunk - Bug #6454]
* test/ruby/test_module.rb (class TestModule):  ditto


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@35759 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information...
commit 1d4e7d923266090844ef1bc1afc8704b75b7d057 1 parent cb61ea5
@drbrain drbrain authored
Showing with 33 additions and 0 deletions.
  1. +6 −0 ChangeLog
  2. +2 −0  class.c
  3. +25 −0 test/ruby/test_module.rb
View
6 ChangeLog
@@ -1,3 +1,9 @@
+Wed May 23 05:15:11 2012 Eric Hodel <drbrain@segment7.net>
+
+ * class.c (rb_mod_init_copy): Clear the cached inspect string of a
+ dup'd anonymous module or class. [ruby-trunk - Bug #6454]
+ * test/ruby/test_module.rb (class TestModule): ditto
+
Tue May 22 16:49:15 2012 Koichi Sasada <ko1@atdot.net>
* vm_core.h: add a data type rb_location_t to store iseq location
View
2  class.c
@@ -174,6 +174,8 @@ rb_mod_init_copy(VALUE clone, VALUE orig)
st_free_table(RCLASS_IV_TBL(clone));
}
RCLASS_IV_TBL(clone) = st_copy(RCLASS_IV_TBL(orig));
+ CONST_ID(id, "__tmp_classpath__");
+ st_delete(RCLASS_IV_TBL(clone), &id, 0);
CONST_ID(id, "__classpath__");
st_delete(RCLASS_IV_TBL(clone), &id, 0);
CONST_ID(id, "__classid__");
View
25 test/ruby/test_module.rb
@@ -259,6 +259,31 @@ def test_constants
assert_equal([:MIXIN, :USER], User.constants.sort)
end
+ def test_dup
+ bug6454 = '[ruby-core:45132]'
+
+ a = Module.new
+ Other.const_set :BUG6454, a
+
+ original = Other::BUG6454.inspect
+
+ b = a.dup
+ Other.const_set :BUG6454_dup, b
+
+ assert_equal "TestModule::Other::BUG6454_dup", b.inspect, bug6454
+ end
+
+ def test_dup_anonymous
+ bug6454 = '[ruby-core:45132]'
+
+ a = Module.new
+ original = a.inspect
+
+ b = a.dup
+
+ refute_equal original, b.inspect, bug6454
+ end
+
def test_included_modules
assert_equal([], Mixin.included_modules)
assert_equal([Mixin], User.included_modules)
Please sign in to comment.
Something went wrong with that request. Please try again.