warnings: separate flags_argmatch

[bison.git] / src / scan-skel.l

diff --git a/src/scan-skel.l b/src/scan-skel.l
index e15294a1e219292be2114417857233c99cb1db8c..68f528cb076d5b3f6fe1d172fad83275a9ae5a00 100644 (file)
--- a/src/scan-skel.l
+++ b/src/scan-skel.l
@@ -41,11 +41,7 @@
 #define YY_DECL static int skel_lex (void)
 YY_DECL;
 
 #define YY_DECL static int skel_lex (void)
 YY_DECL;
 

-#define QPUTS(String) \
-   fputs (quotearg_style (c_quoting_style, String), yyout)
-
-static void at_directive_perform (int at_directive_argc,
-                                  char *at_directive_argv[],
+static void at_directive_perform (int argc, char *argv[],

                                   char **outnamep, int *out_linenop);
 static void fail_for_at_directive_too_many_args (char const *at_directive_name);
 static void fail_for_at_directive_too_few_args (char const *at_directive_name);
                                   char **outnamep, int *out_linenop);
 static void fail_for_at_directive_too_many_args (char const *at_directive_name);
 static void fail_for_at_directive_too_few_args (char const *at_directive_name);


@@ -63,27 +59,25 @@ static void fail_for_invalid_at (char const *at);
 
   /* Currently, only the @warn, @complain, @fatal, @warn_at, @complain_at, and
      @fatal_at directives take multiple arguments, and the last three already
 
   /* Currently, only the @warn, @complain, @fatal, @warn_at, @complain_at, and
      @fatal_at directives take multiple arguments, and the last three already

-     can't take more than 7.  at_directive_argv[0] is the directive name.  */
-  #define AT_DIRECTIVE_ARGC_MAX 8
-  int at_directive_argc = 0;
-  char *at_directive_argv[AT_DIRECTIVE_ARGC_MAX];
+     can't take more than 7.  argv[0] is the directive name.  */
+  #define ARGC_MAX 8
+  int argc = 0;
+  char *argv[ARGC_MAX];

 %}
 
 "@@" fputc ('@', yyout);
 "@{" fputc ('[', yyout);
 "@}" fputc (']', yyout);
 %}
 
 "@@" fputc ('@', yyout);
 "@{" fputc ('[', yyout);
 "@}" fputc (']', yyout);

-"@`" /* Empty.  Used by b4_cat in ../data/bison.m4.  */
-@\n  /* Likewise.  */
+"@`" continue;  /* Used by b4_cat in ../data/bison.m4.  */
+@\n  continue;

 
 "@oline@"  fprintf (yyout, "%d", out_lineno + 1);
 
 "@oline@"  fprintf (yyout, "%d", out_lineno + 1);

-"@ofile@"  QPUTS (outname);
-"@dir_prefix@" QPUTS (dir_prefix);
+"@ofile@"  fputs (quotearg_style (c_quoting_style, outname), yyout);

 
 @[a-z_]+"(" {
   yytext[yyleng-1] = '\0';
   obstack_grow (&obstack_for_string, yytext, yyleng);
 
 @[a-z_]+"(" {
   yytext[yyleng-1] = '\0';
   obstack_grow (&obstack_for_string, yytext, yyleng);

-  at_directive_argv[at_directive_argc++] =
-    obstack_finish (&obstack_for_string);
+  argv[argc++] = obstack_finish (&obstack_for_string);

   BEGIN SC_AT_DIRECTIVE_ARGS;
 }
 
   BEGIN SC_AT_DIRECTIVE_ARGS;
 }
 


@@ -101,49 +95,46 @@ static void fail_for_invalid_at (char const *at);
   return EOF;
 }
 
   return EOF;
 }
 

-<SC_AT_DIRECTIVE_ARGS>{
-  [^@]+ { STRING_GROW; }
+<SC_AT_DIRECTIVE_ARGS>
+{
+  [^@]+  STRING_GROW;

 
 

-  "@@" { obstack_1grow (&obstack_for_string, '@'); }
-  "@{" { obstack_1grow (&obstack_for_string, '['); }
-  "@}" { obstack_1grow (&obstack_for_string, ']'); }
-  "@`" /* Empty.  Useful for starting an argument
-          that begins with whitespace. */
-  @\n  /* Empty.  */
+  "@@"   obstack_1grow (&obstack_for_string, '@');
+  "@{"   obstack_1grow (&obstack_for_string, '[');
+  "@}"   obstack_1grow (&obstack_for_string, ']');
+  "@`"   continue; /* For starting an argument that begins with whitespace. */
+  @\n    continue;

 
   @[,)] {
 
   @[,)] {

-    if (at_directive_argc >= AT_DIRECTIVE_ARGC_MAX)
-      fail_for_at_directive_too_many_args (at_directive_argv[0]);
+    if (argc >= ARGC_MAX)
+      fail_for_at_directive_too_many_args (argv[0]);

 
 

-    obstack_1grow (&obstack_for_string, '\0');
-    at_directive_argv[at_directive_argc++] =
-      obstack_finish (&obstack_for_string);
+    argv[argc++] = obstack_finish0 (&obstack_for_string);

 
     /* Like M4, skip whitespace after a comma.  */
     if (yytext[1] == ',')
       BEGIN SC_AT_DIRECTIVE_SKIP_WS;
     else
       {
 
     /* Like M4, skip whitespace after a comma.  */
     if (yytext[1] == ',')
       BEGIN SC_AT_DIRECTIVE_SKIP_WS;
     else
       {

-        at_directive_perform (at_directive_argc, at_directive_argv,
-                              &outname, &out_lineno);
-        obstack_free (&obstack_for_string, at_directive_argv[0]);
-        at_directive_argc = 0;
+        at_directive_perform (argc, argv, &outname, &out_lineno);
+        obstack_free (&obstack_for_string, argv[0]);
+        argc = 0;

         BEGIN INITIAL;
       }
   }
 
         BEGIN INITIAL;
       }
   }
 

-  @.? { fail_for_invalid_at (yytext); }
+  @.?  fail_for_invalid_at (yytext);

 }
 
 }
 

-<SC_AT_DIRECTIVE_SKIP_WS>{
-  [ \t\r\n]
-  . { yyless (0); BEGIN SC_AT_DIRECTIVE_ARGS; }
+<SC_AT_DIRECTIVE_SKIP_WS>
+{
+  [ \t\r\n]    continue;
+  .            yyless (0); BEGIN SC_AT_DIRECTIVE_ARGS;

 }
 
 }
 

-<SC_AT_DIRECTIVE_ARGS,SC_AT_DIRECTIVE_SKIP_WS>{
-  <<EOF>> {
-    fatal (_("unclosed %s directive in skeleton"), at_directive_argv[0]);
-  }
+<SC_AT_DIRECTIVE_ARGS,SC_AT_DIRECTIVE_SKIP_WS>
+{
+  <<EOF>>  complain (NULL, fatal, _("unclosed %s directive in skeleton"), argv[0]);

 }
 
 %%
 }
 
 %%


@@ -174,132 +165,82 @@ skel_scanner_free (void)
   yylex_destroy ();
 }
 
   yylex_destroy ();
 }
 

-static void
-at_directive_perform (int at_directive_argc,
-                      char *at_directive_argv[],
-                      char **outnamep, int *out_linenop)
+static inline warnings
+flag (const char *arg)

 {
 {

-  if (STREQ (at_directive_argv[0], "@basename"))
+  switch (arg[1])

     {
     {

-      if (at_directive_argc > 2)
-        fail_for_at_directive_too_many_args (at_directive_argv[0]);
-      fputs (last_component (at_directive_argv[1]), yyout);
+    case 'w': return Wother;
+    case 'c': return complaint;
+    case 'f': return fatal;
+    default: aver (false); break;

     }
     }

-  else if (STREQ (at_directive_argv[0], "@warn")
-           || STREQ (at_directive_argv[0], "@complain")
-           || STREQ (at_directive_argv[0], "@fatal"))
+}
+
+static void
+at_directive_perform (int argc, char *argv[], char **outnamep, int *out_linenop)
+{
+  if (STREQ (argv[0], "@basename"))

     {
     {

-      void (*func)(char const *, ...);
-      switch (at_directive_argv[0][1])
-        {
-          case 'w': func = warn; break;
-          case 'c': func = complain; break;
-          case 'f': func = fatal; break;
-          default: aver (false); break;
-        }
-      switch (at_directive_argc)
-        {
-          case 2:
-            func (_(at_directive_argv[1]));
-            break;
-          case 3:
-            func (_(at_directive_argv[1]), at_directive_argv[2]);
-            break;
-          case 4:
-            func (_(at_directive_argv[1]), at_directive_argv[2],
-                  at_directive_argv[3]);
-            break;
-          case 5:
-            func (_(at_directive_argv[1]), at_directive_argv[2],
-                  at_directive_argv[3], at_directive_argv[4]);
-            break;
-          case 6:
-            func (_(at_directive_argv[1]), at_directive_argv[2],
-                  at_directive_argv[3], at_directive_argv[4],
-                  at_directive_argv[5]);
-            break;
-          default:
-            fail_for_at_directive_too_many_args (at_directive_argv[0]);
-            break;
-        }
+      if (argc > 2)
+        fail_for_at_directive_too_many_args (argv[0]);
+      fputs (last_component (argv[1]), yyout);

     }
     }

-  else if (STREQ (at_directive_argv[0], "@warn_at")
-           || STREQ (at_directive_argv[0], "@complain_at")
-           || STREQ (at_directive_argv[0], "@fatal_at"))
+  else if (STREQ (argv[0], "@warn") || STREQ (argv[0], "@warn_at")
+           || STREQ (argv[0], "@complain") || STREQ (argv[0], "@complain_at")
+           || STREQ (argv[0], "@fatal") || STREQ (argv[0], "@fatal_at"))

     {
     {

-      void (*func)(location, char const *, ...);
+      warnings w = flag (*argv);

       location loc;
       location loc;

-      if (at_directive_argc < 4)
-        fail_for_at_directive_too_few_args (at_directive_argv[0]);
-      switch (at_directive_argv[0][1])
-        {
-          case 'w': func = warn_at; break;
-          case 'c': func = complain_at; break;
-          case 'f': func = fatal_at; break;
-          default: aver (false); break;
-        }
-      boundary_set_from_string (&loc.start, at_directive_argv[1]);
-      boundary_set_from_string (&loc.end, at_directive_argv[2]);
-      switch (at_directive_argc)
+      location *locp = NULL;
+      if (STREQ (*argv + strlen (*argv) - 3, "_at"))

         {
         {

-          case 4:
-            func (loc, _(at_directive_argv[3]));
-            break;
-          case 5:
-            func (loc, _(at_directive_argv[3]), at_directive_argv[4]);
-            break;
-          case 6:
-            func (loc, _(at_directive_argv[3]), at_directive_argv[4],
-                  at_directive_argv[5]);
-            break;
-          case 7:
-            func (loc, _(at_directive_argv[3]), at_directive_argv[4],
-                  at_directive_argv[5], at_directive_argv[6]);
-            break;
-          case 8:
-            func (loc, _(at_directive_argv[3]), at_directive_argv[4],
-                  at_directive_argv[5], at_directive_argv[6],
-                  at_directive_argv[7]);
-            break;
-          default:
-            fail_for_at_directive_too_many_args (at_directive_argv[0]);
-            break;
+          if (argc < 4)
+            fail_for_at_directive_too_few_args (argv[0]);
+          boundary_set_from_string (&loc.start, argv[1]);
+          boundary_set_from_string (&loc.end, argv[2]);
+          argc -= 2;
+          argv += 2;
+          locp = &loc;

         }
         }

+      else if (argc < 2)
+        fail_for_at_directive_too_few_args (argv[0]);
+      complain_args (locp, w, argc, argv);

     }
     }

-  else if (STREQ (at_directive_argv[0], "@output"))
+  else if (STREQ (argv[0], "@output"))

     {
     {

-      if (at_directive_argc > 2)
-        fail_for_at_directive_too_many_args (at_directive_argv[0]);
+      if (argc > 2)
+        fail_for_at_directive_too_many_args (argv[0]);

       if (*outnamep)
         {
           free (*outnamep);
           xfclose (yyout);
         }
       if (*outnamep)
         {
           free (*outnamep);
           xfclose (yyout);
         }

-      *outnamep = xstrdup (at_directive_argv[1]);
+      *outnamep = xstrdup (argv[1]);

       output_file_name_check (outnamep);
       yyout = xfopen (*outnamep, "w");
       *out_linenop = 1;
     }
   else
       output_file_name_check (outnamep);
       yyout = xfopen (*outnamep, "w");
       *out_linenop = 1;
     }
   else

-    fail_for_invalid_at (at_directive_argv[0]);
+    fail_for_invalid_at (argv[0]);

 }
 
 static void
 fail_for_at_directive_too_few_args (char const *at_directive_name)
 {
 }
 
 static void
 fail_for_at_directive_too_few_args (char const *at_directive_name)
 {

-  fatal (_("too few arguments for %s directive in skeleton"),
-         at_directive_name);
+  complain (NULL, fatal, _("too few arguments for %s directive in skeleton"),
+            at_directive_name);

 }
 
 static void
 fail_for_at_directive_too_many_args (char const *at_directive_name)
 {
 }
 
 static void
 fail_for_at_directive_too_many_args (char const *at_directive_name)
 {

-  fatal (_("too many arguments for %s directive in skeleton"),
-         at_directive_name);
+  complain (NULL, fatal, _("too many arguments for %s directive in skeleton"),
+            at_directive_name);

 }
 
 static void
 fail_for_invalid_at (char const *at)
 {
 }
 
 static void
 fail_for_invalid_at (char const *at)
 {

-  fatal ("invalid @ in skeleton: %s", at);
+  complain (NULL, fatal, "invalid @ in skeleton: %s", at);

 }
 }