diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-03-24 14:05:45 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-03-24 14:05:45 +0000 |
commit | 0e3822c1e2e81baa96793e881cbbdfc2db7aba3a (patch) | |
tree | 9ae25a4feecf5ac73e5668c84b922180d0aad91a | |
parent | e6a4a8018d1fd070cae7db533840aaa0dbcbe8ec (diff) | |
download | ruby-0e3822c1e2e81baa96793e881cbbdfc2db7aba3a.tar.gz |
* 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-- | ChangeLog | 5 | ||||
-rw-r--r-- | test/ruby/test_defined.rb | 18 | ||||
-rw-r--r-- | variable.c | 2 |
3 files changed, 24 insertions, 1 deletions
@@ -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; } |