From 721b55d70c69aa4cf6386c7876df6ffe0b51ef80 Mon Sep 17 00:00:00 2001 From: nobu Date: Thu, 22 Mar 2018 06:47:53 +0000 Subject: parse.y: reorder in the place * parse.y (new_args_tail): reorder required and optional keyword argument variable IDs in the place. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@62887 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- parse.y | 34 +++++++++++++--------------------- 1 file changed, 13 insertions(+), 21 deletions(-) (limited to 'parse.y') diff --git a/parse.y b/parse.y index 845face90f..f838d3cb05 100644 --- a/parse.y +++ b/parse.y @@ -9963,37 +9963,29 @@ new_args_tail(struct parser_params *p, NODE *kw_args, ID kw_rest_arg, ID block, * #=> * variable order: kr1, k1, k2, internal_id, krest, &b */ - ID kw_bits; + ID kw_bits = internal_id(p), *required_kw_vars, *kw_vars; + struct vtable *vtargs = p->lvtbl->args; NODE *kwn = kw_args; - struct vtable *required_kw_vars = vtable_alloc(NULL); - struct vtable *kw_vars = vtable_alloc(NULL); - int i; + vtable_pop(vtargs, !!block + !!kw_rest_arg); + required_kw_vars = kw_vars = &vtargs->tbl[vtargs->pos]; while (kwn) { - ID vid = kwn->nd_body->nd_vid; + if (!NODE_REQUIRED_KEYWORD_P(kwn->nd_body)) + --kw_vars; + --required_kw_vars; + kwn = kwn->nd_next; + } + for (kwn = kw_args; kwn; kwn = kwn->nd_next) { + ID vid = kwn->nd_body->nd_vid; if (NODE_REQUIRED_KEYWORD_P(kwn->nd_body)) { - vtable_add(required_kw_vars, vid); + *required_kw_vars++ = vid; } else { - vtable_add(kw_vars, vid); + *kw_vars++ = vid; } - - kwn = kwn->nd_next; } - kw_bits = internal_id(p); - - /* reorder */ - vtable_pop(p->lvtbl->args, - vtable_size(required_kw_vars) + vtable_size(kw_vars) - + (block != 0) + (kw_rest_arg != 0)); - - for (i=0; itbl[i]); - for (i=0; itbl[i]); - vtable_free(required_kw_vars); - vtable_free(kw_vars); - arg_var(p, kw_bits); if (kw_rest_arg) arg_var(p, kw_rest_arg); if (block) arg_var(p, block); -- cgit v1.2.3