diff --git a/iseq.c b/iseq.c index 6f16be8525..79b607ee6f 100644 --- a/iseq.c +++ b/iseq.c @@ -1252,8 +1252,8 @@ get_insn_info(const rb_iseq_t *iseq, size_t pos) return &insns_info[i-1]; } -static unsigned int -find_line_no(const rb_iseq_t *iseq, size_t pos) +unsigned int +rb_iseq_line_no(const rb_iseq_t *iseq, size_t pos) { const struct iseq_insn_info_entry *entry = get_insn_info(iseq, pos); @@ -1265,17 +1265,6 @@ find_line_no(const rb_iseq_t *iseq, size_t pos) } } -unsigned int -rb_iseq_line_no(const rb_iseq_t *iseq, size_t pos) -{ - if (pos == 0) { - return find_line_no(iseq, pos); - } - else { - return find_line_no(iseq, pos - 1); - } -} - static VALUE id_to_name(ID id, VALUE default_value) { @@ -1479,8 +1468,8 @@ rb_iseq_disasm_insn(VALUE ret, const VALUE *code, size_t pos, } { - unsigned int line_no = find_line_no(iseq, pos); - unsigned int prev = pos == 0 ? 0 : find_line_no(iseq, pos - 1); + unsigned int line_no = rb_iseq_line_no(iseq, pos); + unsigned int prev = pos == 0 ? 0 : rb_iseq_line_no(iseq, pos - 1); if (line_no && line_no != prev) { long slen = RSTRING_LEN(str); slen = (slen > 70) ? 0 : (70 - slen); @@ -2295,7 +2284,7 @@ rb_iseqw_line_trace_each(VALUE iseqw, int (*func)(int line, rb_event_flag_t *eve trace_num++; if (func) { - int line = find_line_no(iseq, pos); + int line = rb_iseq_line_no(iseq, pos); /* printf("line: %d\n", line); */ cont = (*func)(line, &events, data); if (current_events != events) { diff --git a/vm_backtrace.c b/vm_backtrace.c index f22e3f2944..fd34081667 100644 --- a/vm_backtrace.c +++ b/vm_backtrace.c @@ -31,7 +31,9 @@ id2str(ID id) inline static int calc_lineno(const rb_iseq_t *iseq, const VALUE *pc) { - return rb_iseq_line_no(iseq, pc - iseq->body->iseq_encoded); + size_t pos = (size_t)(pc - iseq->body->iseq_encoded); + /* use pos-1 because PC points next instruction at the beggining of instruction */ + return rb_iseq_line_no(iseq, pos - 1); } int