[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
c.m4: better newline control with b4_parse_param_use
From: |
Akim Demaille |
Subject: |
c.m4: better newline control with b4_parse_param_use |
Date: |
Thu, 15 Mar 2012 17:17:54 +0100 |
This is only is stylistic issue. For master, in a couple
of days.
From 63bf908528f7b8cbfe6dcdff4bd3078ece2179e8 Mon Sep 17 00:00:00 2001
From: Akim Demaille <address@hidden>
Date: Thu, 15 Mar 2012 14:42:15 +0100
Subject: [PATCH 3/6] c.m4: better newline control with b4_parse_param_use.
* data/c.m4: Use m4_ifvaln instead of m4_ifval where
applicable.
(b4_parse_param_use): Switch order between two nested
"if"s to avoid useless empty lines.
Adjust callers to avoid useless lines.
---
data/c.m4 | 17 +++++++----------
data/glr.c | 4 ++--
2 files changed, 9 insertions(+), 12 deletions(-)
diff --git a/data/c.m4 b/data/c.m4
index bc7b4c5..0faf5ad 100644
--- a/data/c.m4
+++ b/data/c.m4
@@ -124,10 +124,8 @@ m4_popdef([$1])dnl
# --------------------------------
# `YYUSE' VAL, LOC if locations are enabled, and all the parse-params.
m4_define([b4_parse_param_use],
-[m4_ifval([$1], [ YYUSE([$1]);
-])dnl
-m4_ifval([$2], [b4_locations_if([ YYUSE ([$2]);
-])])dnl
+[m4_ifvaln([$1], [ YYUSE([$1]);])dnl
+b4_locations_if([m4_ifvaln([$2], [ YYUSE ([$2]);])])dnl
b4_parse_param_for([Decl], [Formal], [ YYUSE (Formal);
])dnl
])
@@ -188,8 +186,7 @@ m4_define([b4_null], [0])
# -------------------------------------------------------------
# Define "yy<TABLE-NAME>" which contents is CONTENT.
m4_define([b4_integral_parser_table_define],
-[m4_ifval([$3], [b4_c_comment([$3], [ ])
-])dnl
+[m4_ifvaln([$3], [b4_c_comment([$3], [ ])])dnl
static const b4_int_type_for([$2]) yy$1[[]] =
{
$2
@@ -448,8 +445,8 @@ m4_define_default([b4_yydestruct_generate],
b4_locations_if( [, [[YYLTYPE *yylocationp], [yylocationp]]])[]dnl
m4_ifset([b4_parse_param], [, b4_parse_param]))[
{
-]b4_parse_param_use([yyvaluep], [yylocationp])[
- if (!yymsg)
+]b4_parse_param_use([yyvaluep], [yylocationp])dnl
+[ if (!yymsg)
yymsg = "Deleting";
YY_SYMBOL_PRINT (yymsg, yytype, yyvaluep, yylocationp);
@@ -483,8 +480,8 @@ m4_define_default([b4_yy_symbol_print_generate],
b4_locations_if([, [[YYLTYPE const * const yylocationp], [yylocationp]]])[]dnl
m4_ifset([b4_parse_param], [, b4_parse_param]))[
{
-]b4_parse_param_use([yyoutput], [yylocationp])[
- if (!yyvaluep)
+]b4_parse_param_use([yyoutput], [yylocationp])dnl
+[ if (!yyvaluep)
return;]
dnl glr.c does not feature yytoknum.
m4_if(b4_skeleton, ["yacc.c"],
diff --git a/data/glr.c b/data/glr.c
index 148af2b..b925868 100644
--- a/data/glr.c
+++ b/data/glr.c
@@ -852,8 +852,8 @@ yyuserAction (yyRuleNum yyn, int yyrhslen, yyGLRStackItem*
yyvsp,
yybool yynormal __attribute__ ((__unused__)) =
(yystackp->yysplitPoint == NULL);
int yylow;
-]b4_parse_param_use([yyvalp], [yylocp])[
-# undef yyerrok
+]b4_parse_param_use([yyvalp], [yylocp])dnl
+[# undef yyerrok
# define yyerrok (yystackp->yyerrState = 0)
# undef YYACCEPT
# define YYACCEPT return yyaccept
--
1.7.9.2
- c.m4: better newline control with b4_parse_param_use,
Akim Demaille <=