diff options
author | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-12-08 13:35:12 +0000 |
---|---|---|
committer | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-12-08 13:35:12 +0000 |
commit | bdb8607cb7549f8555fed781392231ead86d984b (patch) | |
tree | dc4892223043d006fff5b6bf53d9a426d4fa3f60 | |
parent | 537030e19b98cb99d161eff292a33fbdeea19a93 (diff) | |
download | ruby-bdb8607cb7549f8555fed781392231ead86d984b.tar.gz |
* eval.c (top_using): raise a RuntimeError if using is called in a
module definition or a method definition.
* test/ruby/test_refinement.rb: related test.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@38274 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | eval.c | 4 | ||||
-rw-r--r-- | test/ruby/test_refinement.rb | 27 |
3 files changed, 38 insertions, 0 deletions
@@ -1,3 +1,10 @@ +Sat Dec 8 13:17:55 2012 Shugo Maeda <shugo@ruby-lang.org> + + * eval.c (top_using): raise a RuntimeError if using is called in a + module definition or a method definition. + + * test/ruby/test_refinement.rb: related test. + Sat Dec 8 15:01:35 2012 Eric Hodel <drbrain@segment7.net> * lib/rubygems/commands/cleanup_command.rb: Skip default gems when @@ -1393,7 +1393,11 @@ static VALUE top_using(VALUE self, VALUE module) { NODE *cref = rb_vm_cref(); + rb_control_frame_t *prev_cfp = previous_frame(GET_THREAD()); + if (cref->nd_next || (prev_cfp && prev_cfp->me)) { + rb_raise(rb_eRuntimeError, "using is permitted only at toplevel"); + } Check_Type(module, T_MODULE); rb_using_module(cref, module); rb_clear_cache(); diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb index 0d934d737e..016aa15f19 100644 --- a/test/ruby/test_refinement.rb +++ b/test/ruby/test_refinement.rb @@ -624,6 +624,33 @@ class TestRefinement < Test::Unit::TestCase end end + def test_using_in_module + assert_raise(RuntimeError) do + eval(<<-EOF, TOPLEVEL_BINDING) + $main = self + module M + end + module M2 + $main.send(:using, M) + end + EOF + end + end + + def test_using_in_method + assert_raise(RuntimeError) do + eval(<<-EOF, TOPLEVEL_BINDING) + $main = self + module M + end + def call_using_in_method + $main.send(:using, M) + end + call_using_in_method + EOF + end + end + private def eval_using(mod, s) |