From 45ba4645614ab888fc74e66714b4fc7447e07083 Mon Sep 17 00:00:00 2001 From: yugui Date: Thu, 2 Dec 2010 08:05:24 +0000 Subject: merges r29335 from trunk into ruby_1_9_2. -- * string.c (sym_call), vm.c (invoke_block_from_c), vm_insnhelper.c (vm_yield_with_cfunc): pass given block. [ruby-core:32075] * vm_eval.c (rb_funcall_passing_block): new function to call method with passing given block. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_2@30023 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 9 +++++++++ string.c | 4 +++- test/ruby/test_proc.rb | 18 ++++++++++++++++++ version.h | 2 +- vm.c | 1 + vm_eval.c | 8 ++++++++ vm_insnhelper.c | 3 +++ 7 files changed, 43 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1fb131624d..f8d688fa98 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +Fri Sep 24 23:44:59 2010 Nobuyoshi Nakada + + * string.c (sym_call), vm.c (invoke_block_from_c), + vm_insnhelper.c (vm_yield_with_cfunc): pass given block. + [ruby-core:32075] + + * vm_eval.c (rb_funcall_passing_block): new function to call + method with passing given block. + Fri Sep 24 15:50:43 2010 NARUSE, Yui * string.c (rb_str_to_i): fix rdoc: String#to_i raises an diff --git a/string.c b/string.c index c38983561b..fc4f747063 100644 --- a/string.c +++ b/string.c @@ -7171,6 +7171,8 @@ sym_to_sym(VALUE sym) return sym; } +VALUE rb_funcall_passing_block(VALUE recv, ID mid, int argc, const VALUE *argv); + static VALUE sym_call(VALUE args, VALUE sym, int argc, VALUE *argv) { @@ -7180,7 +7182,7 @@ sym_call(VALUE args, VALUE sym, int argc, VALUE *argv) rb_raise(rb_eArgError, "no receiver given"); } obj = argv[0]; - return rb_funcall3(obj, (ID)sym, argc - 1, argv + 1); + return rb_funcall_passing_block(obj, (ID)sym, argc - 1, argv + 1); } /* diff --git a/test/ruby/test_proc.rb b/test/ruby/test_proc.rb index 5a108d3a0f..4a1eee3926 100644 --- a/test/ruby/test_proc.rb +++ b/test/ruby/test_proc.rb @@ -792,4 +792,22 @@ class TestProc < Test::Unit::TestCase assert_equal([obj, nil], [a, b], '[ruby-core:24139]') end end + + def test_block_propagation + bug3792 = '[ruby-core:32075]' + c = Class.new do + def foo + yield + end + end + + o = c.new + f = :foo.to_proc + assert_nothing_raised(LocalJumpError, bug3792) { + assert_equal('bar', f.(o) {'bar'}, bug3792) + } + assert_nothing_raised(LocalJumpError, bug3792) { + assert_equal('zot', o.method(:foo).to_proc.() {'zot'}, bug3792) + } + end end diff --git a/version.h b/version.h index 991ae24b27..5273dc876d 100644 --- a/version.h +++ b/version.h @@ -1,5 +1,5 @@ #define RUBY_VERSION "1.9.2" -#define RUBY_PATCHLEVEL 53 +#define RUBY_PATCHLEVEL 54 #define RUBY_VERSION_MAJOR 1 #define RUBY_VERSION_MINOR 9 #define RUBY_VERSION_TEENY 1 diff --git a/vm.c b/vm.c index f4034b170d..8938df9d51 100644 --- a/vm.c +++ b/vm.c @@ -549,6 +549,7 @@ invoke_block_from_c(rb_thread_t *th, const rb_block_t *block, iseq->local_size - arg_size); ncfp->me = th->passed_me; th->passed_me = 0; + th->passed_block = blockptr; if (cref) { th->cfp->dfp[-1] = (VALUE)cref; diff --git a/vm_eval.c b/vm_eval.c index cadcdb7356..14da1fe5b4 100644 --- a/vm_eval.c +++ b/vm_eval.c @@ -665,6 +665,14 @@ rb_funcall3(VALUE recv, ID mid, int argc, const VALUE *argv) return rb_call(recv, mid, argc, argv, CALL_PUBLIC); } +VALUE +rb_funcall_passing_block(VALUE recv, ID mid, int argc, const VALUE *argv) +{ + PASS_PASSED_BLOCK_TH(GET_THREAD()); + + return rb_call(recv, mid, argc, argv, CALL_PUBLIC); +} + static VALUE send_internal(int argc, const VALUE *argv, VALUE recv, call_type scope) { diff --git a/vm_insnhelper.c b/vm_insnhelper.c index a73fb93714..7dcb4248f1 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -722,6 +722,9 @@ vm_yield_with_cfunc(rb_thread_t *th, const rb_block_t *block, self, (VALUE)block->dfp, 0, th->cfp->sp, block->lfp, 1); + if (blockargptr) { + th->cfp->lfp[0] = GC_GUARDED_PTR((VALUE)blockargptr); + } val = (*ifunc->nd_cfnc) (arg, ifunc->nd_tval, argc, argv, blockarg); th->cfp++; -- cgit v1.2.3