From b443bd081a7db579c4d1206bc95df10b34f18af5 Mon Sep 17 00:00:00 2001 From: yui-knk Date: Sat, 31 May 2014 14:00:30 +0900 Subject: Fix to not print unnecessary comma. When `n2->car` is 0 and `n2->cdd` is not 0, unnecessary comma was printed before first local variables. --- src/parse.y | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/parse.y b/src/parse.y index 9b1d4135f..24cc3a9fe 100644 --- a/src/parse.y +++ b/src/parse.y @@ -5858,6 +5858,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); @@ -5865,8 +5866,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; } @@ -6243,6 +6245,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); @@ -6250,8 +6253,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; } -- cgit v1.2.3