summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2011-03-24 14:05:45 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2011-03-24 14:05:45 +0000
commit0e3822c1e2e81baa96793e881cbbdfc2db7aba3a (patch)
tree9ae25a4feecf5ac73e5668c84b922180d0aad91a
parente6a4a8018d1fd070cae7db533840aaa0dbcbe8ec (diff)
* variable.c (rb_const_defined_0): fix autoloading base.
[ruby-core:35509] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@31169 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog5
-rw-r--r--test/ruby/test_defined.rb18
-rw-r--r--variable.c2
3 files changed, 24 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 554a7f7d93..1472f90787 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Thu Mar 24 23:05:42 2011 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * variable.c (rb_const_defined_0): fix autoloading base.
+ [ruby-core:35509]
+
Thu Mar 24 22:48:43 2011 Tanaka Akira <akr@fsij.org>
* ext/sdbm/_sdbm.c: parenthesize macro arguments.
diff --git a/test/ruby/test_defined.rb b/test/ruby/test_defined.rb
index 9d52e04bc0..e1d685275d 100644
--- a/test/ruby/test_defined.rb
+++ b/test/ruby/test_defined.rb
@@ -85,4 +85,22 @@ class TestDefined < Test::Unit::TestCase
assert_equal 'global-variable', defined?($1)
assert_equal nil, defined?($2)
end
+
+ def test_autoloaded_subclass
+ bug = "[ruby-core:35509]"
+
+ klass = Class.new do
+ autoload(:A, "a")
+ end
+ x = klass.new
+ class << x
+ def a?; defined?(A); end
+ end
+ assert_equal("constant", x.a?, bug)
+
+ klass = Class.new(klass) do
+ def a?; defined?(A); end
+ end
+ assert_equal("constant", klass.new.a?, bug)
+ end
end
diff --git a/variable.c b/variable.c
index 505758c0c0..7f813ad6d4 100644
--- a/variable.c
+++ b/variable.c
@@ -1816,7 +1816,7 @@ rb_const_defined_0(VALUE klass, ID id, int exclude, int recurse, int visibility)
if (visibility && ce->flag == CONST_PRIVATE) {
return (int)Qfalse;
}
- if (ce->value == Qundef && !autoload_node((VALUE)klass, id, 0))
+ if (ce->value == Qundef && !autoload_node(tmp, id, 0))
return (int)Qfalse;
return (int)Qtrue;
}