From 8ed1f433922a45b941fd58e6c3054c8e4c97dd53 Mon Sep 17 00:00:00 2001 From: ko1 Date: Tue, 4 May 2010 03:50:39 +0000 Subject: * vm_insnhelper.c (argument_error): push correct backtrace. Bug #2281 [ruby-core:26333] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@27609 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- vm_insnhelper.c | 37 ++++++++++++++++++++++++++++++------- 1 file changed, 30 insertions(+), 7 deletions(-) (limited to 'vm_insnhelper.c') diff --git a/vm_insnhelper.c b/vm_insnhelper.c index cd74c8e18d..0376b9e46a 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -102,11 +102,37 @@ vm_pop_frame(rb_thread_t *th) /* method dispatch */ +static void +argument_error(const rb_iseq_t *iseq, int miss_argc, int correct_argc) +{ + VALUE mesg = rb_sprintf("wrong number of arguments (%d for %d)", miss_argc, correct_argc); + VALUE exc = rb_exc_new3(rb_eArgError, mesg); + VALUE bt = rb_make_backtrace(); + VALUE err_line = 0; + + if (iseq) { + int line_no = 1; + const char *name; + + if (iseq->insn_info_size) { + line_no = iseq->insn_info_table[0].line_no; + } + + err_line = rb_sprintf("%s:%d:in `%s'", + RSTRING_PTR(iseq->filename), + line_no, RSTRING_PTR(iseq->name)); + rb_funcall(bt, rb_intern("unshift"), 1, err_line); + } + + rb_funcall(exc, rb_intern("set_backtrace"), 1, bt); + rb_exc_raise(exc); +} + #define VM_CALLEE_SETUP_ARG(ret, th, iseq, orig_argc, orig_argv, block) \ if (LIKELY(iseq->arg_simple & 0x01)) { \ /* simple check */ \ if (orig_argc != iseq->argc) { \ - rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", orig_argc, iseq->argc); \ + argument_error(iseq, orig_argc, iseq->argc); \ } \ ret = 0; \ } \ @@ -128,8 +154,7 @@ vm_callee_setup_arg_complex(rb_thread_t *th, const rb_iseq_t * iseq, /* mandatory */ if (argc < (m + iseq->arg_post_len)) { /* check with post arg */ - rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", - argc, m + iseq->arg_post_len); + argument_error(iseq, argc, m + iseq->arg_post_len); } argv += m; @@ -152,8 +177,7 @@ vm_callee_setup_arg_complex(rb_thread_t *th, const rb_iseq_t * iseq, const int opts = iseq->arg_opts - 1 /* no opt */; if (iseq->arg_rest == -1 && argc > opts) { - rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", - orig_argc, m + opts + iseq->arg_post_len); + argument_error(iseq, orig_argc, m + opts + iseq->arg_post_len); } if (argc > opts) { @@ -183,8 +207,7 @@ vm_callee_setup_arg_complex(rb_thread_t *th, const rb_iseq_t * iseq, const rb_block_t *blockptr = *block; if (argc != 0) { - rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", - orig_argc, m + iseq->arg_post_len); + argument_error(iseq, orig_argc, m + iseq->arg_post_len); } if (blockptr) { -- cgit v1.2.3