diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2014-06-25 02:37:39 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2014-06-25 02:37:39 +0900 |
| commit | 9874aafbf087d34730f5b1db763aa88b750d419c (patch) | |
| tree | 21feded29983267d7905d782c41a35851ed92c37 /src/parse.y | |
| parent | 46da47fc5ee8f0e6049d69225114fd93e538ca3a (diff) | |
| parent | 536697f41e7e2844b67f2cc9dac76c14e631af81 (diff) | |
| download | mruby-9874aafbf087d34730f5b1db763aa88b750d419c.tar.gz mruby-9874aafbf087d34730f5b1db763aa88b750d419c.zip | |
Merge pull request #2333 from yui-knk/fix/dump
Fix to not print unnecessary comma.
Diffstat (limited to 'src/parse.y')
| -rw-r--r-- | src/parse.y | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/parse.y b/src/parse.y index 043c86776..0a215f393 100644 --- a/src/parse.y +++ b/src/parse.y @@ -5862,6 +5862,7 @@ mrb_parser_dump(mrb_state *mrb, node *tree, int offset) printf("NODE_SCOPE:\n"); { node *n2 = tree->car; + mrb_bool first_lval = TRUE; if (n2 && (n2->car || n2->cdr)) { dump_prefix(offset+1); @@ -5869,8 +5870,9 @@ mrb_parser_dump(mrb_state *mrb, node *tree, int offset) dump_prefix(offset+2); while (n2) { if (n2->car) { - if (n2 != tree->car) printf(", "); + if (!first_lval) printf(", "); printf("%s", mrb_sym2name(mrb, sym(n2->car))); + first_lval = FALSE; } n2 = n2->cdr; } @@ -6247,6 +6249,7 @@ mrb_parser_dump(mrb_state *mrb, node *tree, int offset) tree = tree->cdr; { node *n2 = tree->car; + mrb_bool first_lval = TRUE; if (n2 && (n2->car || n2->cdr)) { dump_prefix(offset+1); @@ -6254,8 +6257,9 @@ mrb_parser_dump(mrb_state *mrb, node *tree, int offset) dump_prefix(offset+2); while (n2) { if (n2->car) { - if (n2 != tree->car) printf(", "); + if (!first_lval) printf(", "); printf("%s", mrb_sym2name(mrb, sym(n2->car))); + first_lval = FALSE; } n2 = n2->cdr; } |