parse.y: allow arguments start with `_` to appear multiple times; fix #5604

39191cd9 · Yukihiro "Matz" Matsumoto · 3b59c95e · 39191cd9 · 39191cd9
Unverified Commit 39191cd9 authored Dec 24, 2021 by Yukihiro "Matz" Matsumoto
Expand all Hide whitespace changes
Inline Side-by-side

Showing with 1011 additions and 1003 deletions

mrbgems/mruby-compiler/core/parse.y mrbgems/mruby-compiler/core/parse.y +6 -2

mrbgems/mruby-compiler/core/y.tab.c mrbgems/mruby-compiler/core/y.tab.c +1005 -1001

No files found.
--- a/mrbgems/mruby-compiler/core/parse.y
+++ b/mrbgems/mruby-compiler/core/parse.y
@@ -302,8 +302,12 @@ local_add_f(parser_state *p, mrb_sym sym)
    node *n = p->locals->car;
    while (n) {
      if (sym(n->car) == sym) {
-        yyerror(p, "duplicated argument name");
+        mrb_int len;
-        return;
+        const char* name = mrb_sym_name_len(p->mrb, sym, &len);
+        if (len > 0 && name[0] != '_') {
+          yyerror(p, "duplicated argument name");
+          return;
+        }
      }
      n = n->cdr;
    }

--- a/mrbgems/mruby-compiler/core/y.tab.c
+++ b/mrbgems/mruby-compiler/core/y.tab.c