diff options
author | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-11 00:39:04 +0000 |
---|---|---|
committer | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-11 00:39:04 +0000 |
commit | 3360e3e7eafd0a7c19af8dc13b9adba4baee0c48 (patch) | |
tree | deba4b631db5c5b1f811bb2b527f7c7f4f37c4ee | |
parent | e1fd55ae00adc7855f459383a22bb36b4de5ffa5 (diff) |
merges r32334 and r32335 from trunk into ruby_1_9_2.
--
* 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/branches/ruby_1_9_2@32924 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-- | version.h | 2 | ||||
-rw-r--r-- | vm_insnhelper.c | 6 |
5 files changed, 43 insertions, 4 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(). fixes Bug #4613. + + * 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 f885cb0042..ddfcaff920 100644 --- a/test/ruby/test_settracefunc.rb +++ b/test/ruby/test_settracefunc.rb @@ -354,4 +354,26 @@ class TestSetTraceFunc < Test::Unit::TestCase assert_equal([], events[:set]) 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 end @@ -4048,8 +4048,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; } @@ -1,5 +1,5 @@ #define RUBY_VERSION "1.9.2" -#define RUBY_PATCHLEVEL 309 +#define RUBY_PATCHLEVEL 310 #define RUBY_VERSION_MAJOR 1 #define RUBY_VERSION_MINOR 9 #define RUBY_VERSION_TEENY 1 diff --git a/vm_insnhelper.c b/vm_insnhelper.c index 26664ddc13..d760d17e96 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -419,11 +419,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; } |