diff options
author | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-06-30 13:34:53 +0000 |
---|---|---|
committer | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-06-30 13:34:53 +0000 |
commit | 065fe8b7a4dadd41760131cb3778abd4aef26e90 (patch) | |
tree | bdb62730d35f775887284f53e4b952d677cc6be7 | |
parent | c6ef31c9331a364851e805ab35372cd7a9ecbc71 (diff) | |
download | ruby-065fe8b7a4dadd41760131cb3778abd4aef26e90.tar.gz |
* vm_insnhelper.c (vm_call_bmethod): fix to hook call/return event
for methods defined by define_method().
* thread.c (call_trace_proc): Fix to skip if class is not given (0).
Note that ID and Class object are passed for call/return event
if the called method was defined by define_method().
If you are author of tracer/profiler/debugger, this may be an
important change. You should check passed class as zero or
non-zero instead of checking the event type.
* test/ruby/test_settracefunc.rb: add a test for above.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@32334 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 14 | ||||
-rw-r--r-- | test/ruby/test_settracefunc.rb | 22 | ||||
-rw-r--r-- | thread.c | 3 | ||||
-rw-r--r-- | vm_insnhelper.c | 6 |
4 files changed, 42 insertions, 3 deletions
@@ -1,3 +1,17 @@ +Thu Jun 30 22:17:04 2011 Koichi Sasada <ko1@atdot.net> + + * vm_insnhelper.c (vm_call_bmethod): fix to hook call/return event + for methods defined by define_method(). + + * thread.c (call_trace_proc): Fix to skip if class is not given (0). + Note that ID and Class object are passed for call/return event + if the called method was defined by define_method(). + If you are author of tracer/profiler/debugger, this may be an + important change. You should check passed class as zero or + non-zero instead of checking the event type. + + * test/ruby/test_settracefunc.rb: add a test for above. + Thu Jun 30 21:18:35 2011 Yutaka Kanemoto <kanemoto@ruby-lang.org> * configure.in: Add warnflags for XL/C on AIX during configure diff --git a/test/ruby/test_settracefunc.rb b/test/ruby/test_settracefunc.rb index 4bace9ed61..d6c6d06f38 100644 --- a/test/ruby/test_settracefunc.rb +++ b/test/ruby/test_settracefunc.rb @@ -355,6 +355,28 @@ class TestSetTraceFunc < Test::Unit::TestCase assert_equal([], events[:add]) end + def test_trace_defined_method + events = [] + eval <<-EOF.gsub(/^.*?: /, "") + 1: class FooBar; define_method(:foobar){}; end + 2: fb = FooBar.new + 3: set_trace_func(Proc.new { |event, file, lineno, mid, binding, klass| + 4: events << [event, lineno, mid, klass] + 5: }) + 6: fb.foobar + 7: set_trace_func(nil) + EOF + + [["c-return", 5, :set_trace_func, Kernel], + ["line", 6, __method__, self.class], + ["call", 6, :foobar, FooBar], + ["return", 6, :foobar, FooBar], + ["line", 7, __method__, self.class], + ["c-call", 7, :set_trace_func, Kernel]].each{|e| + assert_equal(e, events.shift) + } + end + def test_remove_in_trace bug3921 = '[ruby-dev:42350]' ok = false @@ -4447,8 +4447,7 @@ call_trace_proc(VALUE args, int tracing) ID id = 0; VALUE klass = 0; - if (p->event == RUBY_EVENT_C_CALL || - p->event == RUBY_EVENT_C_RETURN) { + if (p->klass != 0) { id = p->id; klass = p->klass; } diff --git a/vm_insnhelper.c b/vm_insnhelper.c index 366ac4a467..a02628dabb 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -421,11 +421,15 @@ vm_call_bmethod(rb_thread_t *th, VALUE recv, int argc, const VALUE *argv, rb_proc_t *proc; VALUE val; + EXEC_EVENT_HOOK(th, RUBY_EVENT_CALL, recv, me->called_id, me->klass); + /* control block frame */ th->passed_me = me; - GetProcPtr(me->def->body.proc, proc); val = rb_vm_invoke_proc(th, proc, recv, argc, argv, blockptr); + + EXEC_EVENT_HOOK(th, RUBY_EVENT_RETURN, recv, me->called_id, me->klass); + return val; } |