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.

diff --git a/ChangeLog b/ChangeLog
index bfa5eee395c094b10b034297ba0880a2e35b6832..db9bf08c305fe92bfcff4bf11cf509ccb36f1768 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -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>
 
        * tests/c++.at (AT_CHECK_VARIANTS): Fixes tests 198-202.
 2008-11-17  Di-an Jan  <dianj@freeshell.org>
 
        * tests/c++.at (AT_CHECK_VARIANTS): Fixes tests 198-202.

diff --git a/src/getargs.c b/src/getargs.c
index 53934f336084a3a2b6de972d0a7ffdb53c01cfe4..6b748c8b421ce934882f5e9a2aa2aee46e46a8aa 100644 (file)
--- a/src/getargs.c
+++ b/src/getargs.c
@@ -499,7 +499,7 @@ static struct option const long_options[] =
 /* Build a location for the current command line argument. */
 static
 location
 /* Build a location for the current command line argument. */
 static
 location

-command_line_location()
+command_line_location (void)

 {
   location res;
   /* "<command line>" is used in GCC's messages about -D. */
 {
   location res;
   /* "<command line>" is used in GCC's messages about -D. */

diff --git a/src/output.c b/src/output.c
index 31bfd12d94310c4f2ee5ed930096b2c80370c337..73e634855e6c2a4a184e2e2b2d9e38a4a6d4a493 100644 (file)
--- a/src/output.c
+++ b/src/output.c
@@ -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)
 {
   int res = UNIQSTR_CMP((*lhs)->type_name, (*rhs)->type_name);
 symbol_type_name_cmp (const symbol **lhs, const symbol **rhs)
 {
   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 *);
   symbol **res = xmemdup (symbols, nsyms * sizeof *res);
 {
   typedef int (*qcmp_type) (const void *, const void *);
   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;
   for (i = 0; i < nsyms; ++i)
 {
   int i;
   for (i = 0; i < nsyms; ++i)


@@ -596,7 +596,6 @@ muscles_output (FILE *out)
   merger_output (out);
   symbol_code_props_output (out, "destructors", &symbol_destructor_get);
   symbol_code_props_output (out, "printers", &symbol_printer_get);
   merger_output (out);
   symbol_code_props_output (out, "destructors", &symbol_destructor_get);
   symbol_code_props_output (out, "printers", &symbol_printer_get);

-  symbol_definitions_output (out);

   symbol_numbers_output (out);
   token_definitions_output (out);
   type_names_output (out);
   symbol_numbers_output (out);
   token_definitions_output (out);
   type_names_output (out);


@@ -794,6 +793,7 @@ output (void)
   prepare_rules ();
   prepare_states ();
   prepare_actions ();
   prepare_rules ();
   prepare_states ();
   prepare_actions ();

+  prepare_symbol_definitions ();

 
   prepare ();
 
 
   prepare ();
 

diff --git a/tests/c++.at b/tests/c++.at
index f1b1b3ac9fda3173dc800640ddea6478ba72bbbd..e2f9d03f400b892b1cc8b7f78b3fdefccc870e23 100644 (file)
--- a/tests/c++.at
+++ b/tests/c++.at
@@ -174,7 +174,7 @@ yy::parser::error(const yy::parser::location_type&,
 }
 
 int
 }
 
 int

-main(int argc, char *argv[])
+main (void)

 {
   yy::parser p;
   p.set_debug_level(!!getenv("YYDEBUG"));
 {
   yy::parser p;
   p.set_debug_level(!!getenv("YYDEBUG"));


@@ -310,7 +310,7 @@ AT_DATA_GRAMMAR([[input.y]],
 
 %code {
   // YYSTYPE contains a namespace reference.
 
 %code {
   // 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;
     return 0;
   }
     lval->i = 3;
     return 0;
   }