Future/bison
1
0
Fork 0
mirror of https://git.savannah.gnu.org/git/bison.git synced 2026-03-14 06:43:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Handles several --enable-gcc-warnings.

Browse Source 
* src/getargs.c (command_line_location): Set parameters to void.
* src/output.c (symbol_type_name_cmp): Make static.
(symbols_by_type_name): Set parameters to void.
(symbol_definitions_output): Remove unused parameter.  Rename as...
(prepare_symbol_definitions): this.
(muscles_output): Move symbol_definitions_output to...
(output): here as prepare_symbol_definitions.
* tests/c++.at (AT_CHECK_VARIANTS): Remove unused parameters of main.
(AT_CHECK_NAMESPACE): Make unused parameter lloc unnamed.
This commit is contained in:
Di-an Jan
2008-11-17 11:01:41 +01:00
committed by Paolo Bonzini
parent 68c989dea9
commit d73e55e022
4 changed files with 23 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
ChangeLog
View File

@@ -1,3 +1,16 @@
2008-11-17 Di-an Jan <dianj@freeshell.org>
Handles several --enable-gcc-warnings.
* src/getargs.c (command_line_location): Set parameters to void.
* src/output.c (symbol_type_name_cmp): Make static.
(symbols_by_type_name): Set parameters to void.
(symbol_definitions_output): Remove unused parameter. Rename as...
(prepare_symbol_definitions): this.
(muscles_output): Move symbol_definitions_output to...
(output): here as prepare_symbol_definitions.
* tests/c++.at (AT_CHECK_VARIANTS): Remove unused parameters of main.
(AT_CHECK_NAMESPACE): Make unused parameter lloc unnamed.
2008-11-17 Di-an Jan <dianj@freeshell.org> 2008-11-17 Di-an Jan <dianj@freeshell.org>
* tests/c++.at (AT_CHECK_VARIANTS): Fixes tests 198-202. * tests/c++.at (AT_CHECK_VARIANTS): Fixes tests 198-202.

2
src/getargs.c
View File

@@ -499,7 +499,7 @@ static struct option const long_options[] =
/* Build a location for the current command line argument. */ /* Build a location for the current command line argument. */
static static
location location
command_line_location() command_line_location (void)
{ {
location res; location res;
/* "<command line>" is used in GCC's messages about -D. */ /* "<command line>" is used in GCC's messages about -D. */

14
src/output.c
View File

@@ -287,7 +287,7 @@ prepare_states (void)
| Compare two symbols by type-name, and then by number. | | Compare two symbols by type-name, and then by number. |
`-------------------------------------------------------*/ `-------------------------------------------------------*/
int static int
symbol_type_name_cmp (const symbol **lhs, const symbol **rhs) symbol_type_name_cmp (const symbol **lhs, const symbol **rhs)
{ {
int res = UNIQSTR_CMP((*lhs)->type_name, (*rhs)->type_name); int res = UNIQSTR_CMP((*lhs)->type_name, (*rhs)->type_name);
@@ -302,7 +302,7 @@ symbol_type_name_cmp (const symbol **lhs, const symbol **rhs)
`----------------------------------------------------------------*/ `----------------------------------------------------------------*/
static symbol ** static symbol **
symbols_by_type_name () symbols_by_type_name (void)
{ {
typedef int (*qcmp_type) (const void *, const void *); typedef int (*qcmp_type) (const void *, const void *);
symbol **res = xmemdup (symbols, nsyms * sizeof *res); symbol **res = xmemdup (symbols, nsyms * sizeof *res);
@@ -396,12 +396,12 @@ merger_output (FILE *out)
} }
/*---------------------------------------. /*---------------------------------------------.
| Output the symbol definitions to OUT. | | Prepare the muscles for symbol definitions. |
`---------------------------------------*/ `---------------------------------------------*/
static void static void
symbol_definitions_output (FILE *out) prepare_symbol_definitions (void)
{ {
int i; int i;
for (i = 0; i < nsyms; ++i) for (i = 0; i < nsyms; ++i)
@@ -596,7 +596,6 @@ muscles_output (FILE *out)
merger_output (out); merger_output (out);
symbol_code_props_output (out, "destructors", &symbol_destructor_get); symbol_code_props_output (out, "destructors", &symbol_destructor_get);
symbol_code_props_output (out, "printers", &symbol_printer_get); symbol_code_props_output (out, "printers", &symbol_printer_get);
symbol_definitions_output (out);
symbol_numbers_output (out); symbol_numbers_output (out);
token_definitions_output (out); token_definitions_output (out);
type_names_output (out); type_names_output (out);
@@ -794,6 +793,7 @@ output (void)
prepare_rules (); prepare_rules ();
prepare_states (); prepare_states ();
prepare_actions (); prepare_actions ();
prepare_symbol_definitions ();
prepare (); prepare ();

4
tests/c++.at
View File

@@ -174,7 +174,7 @@ yy::parser::error(const yy::parser::location_type&,
} }
int int
main(int argc, char *argv[]) main (void)
{ {
yy::parser p; yy::parser p;
p.set_debug_level(!!getenv("YYDEBUG")); p.set_debug_level(!!getenv("YYDEBUG"));
@@ -310,7 +310,7 @@ AT_DATA_GRAMMAR([[input.y]],
%code { %code {
// YYSTYPE contains a namespace reference. // YYSTYPE contains a namespace reference.
int yylex (YYSTYPE *lval, const ]$1[::parser::location_type* lloc) { int yylex (YYSTYPE *lval, const ]$1[::parser::location_type*) {
lval->i = 3; lval->i = 3;
return 0; return 0;
} }