style changes

* data/glr.cc, tests/actions.at: Fix comments.
* src/symtab.h, src/symtab.c: Fix indentation/comments.
* src/symlist.c: Fix indentation.
This commit is contained in:
Akim Demaille
2012-08-14 10:04:27 +02:00
parent 1ad6f4daa3
commit 3edfae0490
5 changed files with 24 additions and 24 deletions

View File

@@ -62,7 +62,7 @@ m4_define([b4_parse_param_orig], m4_defn([b4_parse_param]))
# b4_yy_symbol_print_define # b4_yy_symbol_print_define
# --------------------------- # -------------------------
# Bypass the default implementation to generate the "yy_symbol_print" # Bypass the default implementation to generate the "yy_symbol_print"
# and "yy_symbol_value_print" functions. # and "yy_symbol_value_print" functions.
m4_define([b4_yy_symbol_print_define], m4_define([b4_yy_symbol_print_define],
@@ -123,7 +123,7 @@ m4_append([b4_epilogue],
]b4_namespace_open[ ]b4_namespace_open[
]dnl In this section, the parse param are the original parse_params. ]dnl In this section, the parse params are the original parse_params.
m4_pushdef([b4_parse_param], m4_defn([b4_parse_param_orig]))dnl m4_pushdef([b4_parse_param], m4_defn([b4_parse_param_orig]))dnl
[ /// Build a parser object. [ /// Build a parser object.
]b4_parser_class_name::b4_parser_class_name[ (]b4_parse_param_decl[)]m4_ifset([b4_parse_param], [ ]b4_parser_class_name::b4_parser_class_name[ (]b4_parse_param_decl[)]m4_ifset([b4_parse_param], [

View File

@@ -195,18 +195,18 @@ symbol_list_code_props_set (symbol_list *node, code_props_type kind,
{ {
switch (node->content_type) switch (node->content_type)
{ {
case SYMLIST_SYMBOL: case SYMLIST_SYMBOL:
symbol_code_props_set (node->content.sym, kind, cprops); symbol_code_props_set (node->content.sym, kind, cprops);
if (node->content.sym->status == undeclared) if (node->content.sym->status == undeclared)
node->content.sym->status = used; node->content.sym->status = used;
break; break;
case SYMLIST_TYPE: case SYMLIST_TYPE:
semantic_type_code_props_set semantic_type_code_props_set
(semantic_type_get (node->content.sem_type->tag, (semantic_type_get (node->content.sem_type->tag,
&node->content.sem_type->location), &node->content.sem_type->location),
kind, cprops); kind, cprops);
if (node->content.sem_type->status == undeclared) if (node->content.sem_type->status == undeclared)
node->content.sem_type->status = used; node->content.sem_type->status = used;
break; break;
} }
} }

View File

@@ -396,7 +396,7 @@ symbol_check_defined (symbol *sym)
complain_at (sym->location, complaint, complain_at (sym->location, complaint,
_("symbol %s is used, but is not defined as a token" _("symbol %s is used, but is not defined as a token"
" and has no rules"), " and has no rules"),
sym->tag); sym->tag);
break; break;
case declared: case declared:
/* If declared, then sym->class != unknown_sym. */ /* If declared, then sym->class != unknown_sym. */

View File

@@ -54,13 +54,12 @@ typedef struct symbol symbol;
/* Declaration status of a symbol. /* Declaration status of a symbol.
First, it is "undeclared". Then, if "undeclared" and used in a First, it is "undeclared". Then, if "undeclared" and used in a
%printer/%destructor, it is "used". If not "declared" by used in a %printer/%destructor, it is "used". If not "declared" but used in
rule, it is "needed". Finally, if declared (via a rule for a rule, it is "needed". Finally, if declared (via a rule for
nonterminals, or %oken), it is "declared". nonterminals, or %token), it is "declared".
When status are checked at the end, "declared" symbols are fine, When status are checked at the end, "declared" symbols are fine,
"used" symbols trigger warnings, otherwise it's an error. "used" symbols trigger warnings, otherwise it's an error. */
*/
typedef enum typedef enum
{ {

View File

@@ -1259,10 +1259,11 @@ AT_CHECK_ACTION_LOCATIONS([[%printer]])
## Qualified $$ in actions. ## ## Qualified $$ in actions. ##
## ------------------------- ## ## ------------------------- ##
# Check that we can used qualified $$ (v.g., $<type>$) not only in # Check that we can use qualified $$ (v.g., $<type>$) not only in rule
# rule actions, but also where $$ is valid: %printer and %destructor. # actions, but also where $$ is valid: %destructor/%printer and
# %initial-action.
# #
# FIXME: Not actually checking %desctructor, but it's the same code as # FIXME: Not actually checking %destructor, but it's the same code as
# %printer... # %printer...
# #
# To do that, use a semantic value that has two fields (sem_type), # To do that, use a semantic value that has two fields (sem_type),