From 2e83c555d83d379f65f8e6af79ce7bda36a7ba2c Mon Sep 17 00:00:00 2001 From: ko1 Date: Sat, 18 Aug 2007 04:17:39 +0000 Subject: * insnhelper.ci (vm_callee_setup_arg): fix to check arguments correctly. [ruby-dev:31472] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@13090 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- insnhelper.ci | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'insnhelper.ci') diff --git a/insnhelper.ci b/insnhelper.ci index 2922163733..17c4047798 100644 --- a/insnhelper.ci +++ b/insnhelper.ci @@ -103,7 +103,7 @@ vm_callee_setup_arg(rb_thread_t *th, rb_iseq_t *iseq, const int m = iseq->argc; const int orig_argc = argc; - if (iseq->arg_simple & 0x01) { + if (LIKELY(iseq->arg_simple & 0x01)) { /* simple check */ if (argc != m) { rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", @@ -136,7 +136,7 @@ vm_callee_setup_arg(rb_thread_t *th, rb_iseq_t *iseq, for (i=0; iarg_post_len; i++) { dst[iseq->arg_post_start + iseq->arg_post_len - (i + 1)] = argv[argc - 1]; - argc = argc - 1; + argc--; } } @@ -162,6 +162,7 @@ vm_callee_setup_arg(rb_thread_t *th, rb_iseq_t *iseq, /* rest arguments */ if (iseq->arg_rest != -1) { dst[iseq->arg_rest] = rb_ary_new4(argc, argv); + argc = 0; } /* block arguments */ @@ -169,6 +170,11 @@ vm_callee_setup_arg(rb_thread_t *th, rb_iseq_t *iseq, VALUE blockval = Qnil; rb_block_t * const blockptr = *block; + if (argc != 0) { + rb_raise(rb_eArgError, "wrong number of arguments (%d for %d)", + orig_argc, m + iseq->arg_post_len); + } + if (blockptr) { /* make Proc object */ if (blockptr->proc == 0) { -- cgit v1.2.3