From 6e0dd3e7c1786e33c3193d0b4dd9579655c4e842 Mon Sep 17 00:00:00 2001 From: Takashi Kokubun Date: Fri, 20 Sep 2019 21:06:07 +0900 Subject: [PATCH] Use RUBY_VM_NEXT_CONTROL_FRAME macro in vm_push_frame and limit scope of i. Just a minor maintainability improvement. --- vm_insnhelper.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/vm_insnhelper.c b/vm_insnhelper.c index bea8d10037..56767a4a62 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -269,8 +269,7 @@ vm_push_frame(rb_execution_context_t *ec, int local_size, int stack_max) { - rb_control_frame_t *const cfp = ec->cfp - 1; - int i; + rb_control_frame_t *const cfp = RUBY_VM_NEXT_CONTROL_FRAME(ec->cfp); vm_check_frame(type, specval, cref_or_me, iseq); VM_ASSERT(local_size >= 0); @@ -290,7 +289,7 @@ vm_push_frame(rb_execution_context_t *ec, /* setup vm value stack */ /* initialize local variables */ - for (i=0; i < local_size; i++) { + for (int i=0; i < local_size; i++) { *sp++ = Qnil; }