Module Name: src
Committed By: rillig
Date: Sun Jun 27 21:36:14 UTC 2021
Modified Files:
src/usr.bin/xlint/lint1: cgram.y
Log Message:
lint: reorder grammar rules from big to small
No functional change.
To generate a diff of this commit:
cvs rdiff -u -r1.238 -r1.239 src/usr.bin/xlint/lint1/cgram.y
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/usr.bin/xlint/lint1/cgram.y
diff -u src/usr.bin/xlint/lint1/cgram.y:1.238 src/usr.bin/xlint/lint1/cgram.y:1.239
--- src/usr.bin/xlint/lint1/cgram.y:1.238 Sun Jun 27 21:30:46 2021
+++ src/usr.bin/xlint/lint1/cgram.y Sun Jun 27 21:36:14 2021
@@ -1,5 +1,5 @@
%{
-/* $NetBSD: cgram.y,v 1.238 2021/06/27 21:30:46 rillig Exp $ */
+/* $NetBSD: cgram.y,v 1.239 2021/06/27 21:36:14 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.238 2021/06/27 21:30:46 rillig Exp $");
+__RCSID("$NetBSD: cgram.y,v 1.239 2021/06/27 21:36:14 rillig Exp $");
#endif
#include <limits.h>
@@ -1075,7 +1075,15 @@ notype_decl:
}
;
-/* TODO: move below type_decl */
+type_decl:
+ type_direct_decl {
+ $$ = $1;
+ }
+ | pointer type_direct_decl {
+ $$ = add_pointer($2, $1);
+ }
+ ;
+
notype_direct_decl:
T_NAME {
$$ = declarator_name(getsym($1));
@@ -1101,18 +1109,9 @@ notype_direct_decl:
/* TODO: either add { $$ = $1 } everywhere or remove it everywhere. */
;
-type_decl:
- type_direct_decl {
- $$ = $1;
- }
- | pointer type_direct_decl {
- $$ = add_pointer($2, $1);
- }
- ;
-
/*
- * TODO: document whether T_NAME here vs. identifier in notype_direct_decl
- * is on purpose.
+ * TODO: document whether the subtle difference between 'identifier' here and
+ * 'T_NAME' in notype_direct_decl above is on purpose.
*/
type_direct_decl:
identifier {
@@ -1415,19 +1414,20 @@ initializer_list_item:
| initializer
;
-range:
- constant_expr {
- $$.lo = to_int_constant($1, true);
- $$.hi = $$.lo;
- }
- | constant_expr T_ELLIPSIS constant_expr {
- $$.lo = to_int_constant($1, true);
- $$.hi = to_int_constant($3, true);
- /* initialization with '[a...b]' is a GNU extension */
- gnuism(340);
+designation: /* C99 6.7.8 "Initialization" */
+ designator_list T_ASSIGN
+ | identifier T_COLON {
+ /* GCC style struct or union member name in initializer */
+ gnuism(315);
+ add_designator_member($1);
}
;
+designator_list: /* C99 6.7.8 "Initialization" */
+ designator
+ | designator_list designator
+ ;
+
designator: /* C99 6.7.8 "Initialization" */
T_LBRACK range T_RBRACK {
add_designator_subscript($2);
@@ -1443,18 +1443,16 @@ designator: /* C99 6.7.8 "Initializati
}
;
-designator_list: /* C99 6.7.8 "Initialization" */
- designator
- | designator_list designator
- ;
-
-/* TODO: order from big to small */
-designation: /* C99 6.7.8 "Initialization" */
- designator_list T_ASSIGN
- | identifier T_COLON {
- /* GCC style struct or union member name in initializer */
- gnuism(315);
- add_designator_member($1);
+range:
+ constant_expr {
+ $$.lo = to_int_constant($1, true);
+ $$.hi = $$.lo;
+ }
+ | constant_expr T_ELLIPSIS constant_expr {
+ $$.lo = to_int_constant($1, true);
+ $$.hi = to_int_constant($3, true);
+ /* initialization with '[a...b]' is a GNU extension */
+ gnuism(340);
}
;