Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/xlint/lint1 lint: remove redundant '{ $$ = $1; }' fr...



details:   https://anonhg.NetBSD.org/src/rev/09ac91bd60d0
branches:  trunk
changeset: 1021964:09ac91bd60d0
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Jun 27 21:46:17 2021 +0000

description:
lint: remove redundant '{ $$ = $1; }' from the grammar

In some cases the redundant code had been spelled out, in others it had
been left out.  Be consistent.

diffstat:

 usr.bin/xlint/lint1/cgram.y |  101 +++++++++++--------------------------------
 1 files changed, 26 insertions(+), 75 deletions(-)

diffs (283 lines):

diff -r ff1678004882 -r 09ac91bd60d0 usr.bin/xlint/lint1/cgram.y
--- a/usr.bin/xlint/lint1/cgram.y       Sun Jun 27 21:36:14 2021 +0000
+++ b/usr.bin/xlint/lint1/cgram.y       Sun Jun 27 21:46:17 2021 +0000
@@ -1,5 +1,5 @@
 %{
-/* $NetBSD: cgram.y,v 1.239 2021/06/27 21:36:14 rillig Exp $ */
+/* $NetBSD: cgram.y,v 1.240 2021/06/27 21:46:17 rillig Exp $ */
 
 /*
  * Copyright (c) 1996 Christopher G. Demetriou.  All Rights Reserved.
@@ -35,7 +35,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: cgram.y,v 1.239 2021/06/27 21:36:14 rillig Exp $");
+__RCSID("$NetBSD: cgram.y,v 1.240 2021/06/27 21:46:17 rillig Exp $");
 #endif
 
 #include <limits.h>
@@ -702,9 +702,7 @@
        ;
 
 typespec:
-         notype_typespec {
-               $$ = $1;
-         }
+         notype_typespec
        | T_TYPENAME {
                $$ = getsym($1)->s_type;
          }
@@ -784,9 +782,7 @@
        ;
 
 member_declaration_list_with_rbrace:
-         member_declaration_list T_SEMI T_RBRACE {
-               $$ = $1;
-         }
+         member_declaration_list T_SEMI T_RBRACE
        | member_declaration_list T_RBRACE {
                if (sflag) {
                        /* syntax req. ';' after last struct/union member */
@@ -808,9 +804,7 @@
        ;
 
 member_declaration_list:
-         member_declaration {
-               $$ = $1;
-         }
+         member_declaration
        | member_declaration_list T_SEMI member_declaration {
                $$ = lnklst($1, $3);
          }
@@ -911,9 +905,7 @@
        ;
 
 notype_member_decl:
-         notype_decl {
-               $$ = $1;
-         }
+         notype_decl
        | notype_decl T_COLON constant_expr {           /* C99 6.7.2.1 */
                $$ = bitfield($1, to_int_constant($3, true));
          }
@@ -925,9 +917,7 @@
        ;
 
 type_member_decl:
-         type_decl {
-               $$ = $1;
-         }
+         type_decl
        | type_decl T_COLON constant_expr {
                $$ = bitfield($1, to_int_constant($3, true));
          }
@@ -985,9 +975,7 @@
        ;
 
 enums_with_opt_comma:
-         enums {
-               $$ = $1;
-         }
+         enums
        | enums T_COMMA {
                if (sflag) {
                        /* trailing ',' prohibited in enum declaration */
@@ -1001,9 +989,7 @@
        ;
 
 enums:
-         enumerator {
-               $$ = $1;
-         }
+         enumerator
        | enums T_COMMA enumerator {
                $$ = lnklst($1, $3);
          }
@@ -1067,18 +1053,14 @@
        ;
 
 notype_decl:
-         notype_direct_decl {
-               $$ = $1;
-         }
+         notype_direct_decl
        | pointer notype_direct_decl {
                $$ = add_pointer($2, $1);
          }
        ;
 
 type_decl:
-         type_direct_decl {
-               $$ = $1;
-         }
+         type_direct_decl
        | pointer type_direct_decl {
                $$ = add_pointer($2, $1);
          }
@@ -1106,7 +1088,6 @@
                block_level--;
          }
        | notype_direct_decl type_attribute_list
-/* TODO: either add { $$ = $1 } everywhere or remove it everywhere. */
        ;
 
 /*
@@ -1145,9 +1126,7 @@
  *                             not "typedef int a; f(int a);"
  */
 param_decl:
-         direct_param_decl {
-               $$ = $1;
-         }
+         direct_param_decl
        | pointer direct_param_decl {
                $$ = add_pointer($2, $1);
          }
@@ -1168,9 +1147,7 @@
                c11ism(343);
                $$ = $3;
          }
-       | constant_expr {
-               $$ = $1;
-       }
+       | constant_expr
        ;
 
 direct_param_decl:
@@ -1197,9 +1174,7 @@
        ;
 
 notype_param_decl:
-         direct_notype_param_decl {
-               $$ = $1;
-         }
+         direct_notype_param_decl
        | pointer direct_notype_param_decl {
                $$ = add_pointer($2, $1);
          }
@@ -1227,9 +1202,7 @@
 
 /* TODO: rename 'pointer' to something less ambiguous, maybe 'pointer_level' */
 pointer:
-         asterisk {
-               $$ = $1;
-         }
+         asterisk
        | asterisk type_qualifier_list {
                /* TODO: rename pqinf_t to be more expressive */
                /* TODO: then rename the merge function */
@@ -1253,9 +1226,7 @@
 
 /* TODO: try whether type_qualifier_list_opt makes the code simpler */
 type_qualifier_list:
-         type_qualifier {
-               $$ = $1;
-         }
+         type_qualifier
        | type_qualifier_list type_qualifier {
                $$ = merge_pointers_and_qualifiers($1, $2);
          }
@@ -1278,9 +1249,7 @@
          id_list_lparen identifier_list T_RPAREN {
                $$ = $2;
          }
-       | abstract_decl_param_list {
-               $$ = $1;
-         }
+       | abstract_decl_param_list
        ;
 
 id_list_lparen:
@@ -1297,9 +1266,7 @@
        | identifier_list T_COMMA T_NAME {
                $$ = lnklst($1, old_style_function_name(getsym($3)));
          }
-       | identifier_list error {
-               $$ = $1;
-         }
+       | identifier_list error
        ;
 
 abstract_decl_param_list:
@@ -1323,9 +1290,7 @@
        ;
 
 vararg_parameter_type_list:
-         parameter_type_list {
-               $$ = $1;
-         }
+         parameter_type_list
        | parameter_type_list T_COMMA T_ELLIPSIS {
                dcs->d_vararg = true;
                $$ = $1;
@@ -1344,9 +1309,7 @@
        ;
 
 parameter_type_list:
-         parameter_declaration {
-               $$ = $1;
-         }
+         parameter_declaration
        | parameter_type_list T_COMMA parameter_declaration {
                $$ = lnklst($1, $3);
          }
@@ -1497,9 +1460,7 @@
          pointer {
                $$ = add_pointer(abstract_name(), $1);
          }
-       | direct_abstract_decl {
-               $$ = $1;
-         }
+       | direct_abstract_decl
        | pointer direct_abstract_decl {
                $$ = add_pointer($2, $1);
          }
@@ -1791,9 +1752,7 @@
          /* empty */ {
                $$ = NULL;
          }
-       | expr {
-               $$ = $1;
-         }
+       | expr
        ;
 
 jump_statement:                        /* C99 6.8.6 */
@@ -1850,9 +1809,7 @@
        ;
 
 constant_expr:                 /* C99 6.6 */
-         expr                          %prec T_ASSIGN {
-                 $$ = $1;
-         }
+         expr %prec T_ASSIGN
        ;
 
 expr:
@@ -1901,12 +1858,8 @@
        | expr T_COMMA expr {
                $$ = build(COMMA, $1, $3);
          }
-       | term {
-               $$ = $1;
-         }
-       | generic_selection {
-               $$ = $1;
-         }
+       | term
+       | generic_selection
        ;
 
 assignment_expression:         /* C99 6.5.16 */
@@ -2077,9 +2030,7 @@
        ;
 
 string:
-         T_STRING {
-               $$ = $1;
-         }
+         T_STRING
        | T_STRING string2 {
                $$ = cat_strings($1, $2);
          }



Home | Main Index | Thread Index | Old Index