From 58ba24f8858ed4e9cae3dfe5aa8e62e0c6316878 Mon Sep 17 00:00:00 2001 From: ko1 Date: Thu, 18 Jun 2015 09:01:00 +0000 Subject: [PATCH] * vm.c (rb_vm_control_frame_id_and_class): remove usless codes. `me' knows ID and owner class. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50950 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ vm.c | 23 ++++------------------- 2 files changed, 9 insertions(+), 19 deletions(-) diff --git a/ChangeLog b/ChangeLog index ee2e45b58a..087b1e3442 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Thu Jun 18 17:59:06 2015 Koichi Sasada + + * vm.c (rb_vm_control_frame_id_and_class): remove usless codes. + `me' knows ID and owner class. + Thu Jun 18 16:58:35 2015 Koichi Sasada * method.h: constify rb_method_alias_struct::original_me and diff --git a/vm.c b/vm.c index 006b0f8a80..69b594f950 100644 --- a/vm.c +++ b/vm.c @@ -1738,31 +1738,16 @@ rb_iseq_eval_main(VALUE iseqval) int rb_vm_control_frame_id_and_class(const rb_control_frame_t *cfp, ID *idp, VALUE *klassp) { - rb_iseq_t *iseq = cfp->iseq; const rb_method_entry_t *me = rb_vm_frame_method_entry(cfp); - if (!iseq && me) { /* TODO: me should know all */ + if (me) { if (idp) *idp = me->def->original_id; if (klassp) *klassp = me->klass; - return 1; + return TRUE; } - while (iseq) { - if (RUBY_VM_IFUNC_P(iseq)) { - if (idp) *idp = idIFUNC; - if (klassp) *klassp = 0; - return 1; - } - if (iseq->defined_method_id) { - if (idp) *idp = iseq->defined_method_id; - if (klassp) *klassp = iseq->klass; - return 1; - } - if (iseq->local_iseq == iseq) { - break; - } - iseq = iseq->parent_iseq; + else { + return FALSE; } - return 0; } int