Re: Bison-2.1 grl :)

[Paul Eggert]

address@hidden writes:

>> Hmm, I can't reproduce this problem.
> see the gcc -Wswitch-default option
> and when using that option should give warning.
> then adding default statement should solve.

Ah, thanks, I'd forgotten about that one.  I installed this:

2005-09-15  Paul Eggert  <address@hidden>

        * data/glr.c (yyresolveValue): Add default case to pacify
        gcc -Wswitch-default.  Problem reported by address@hidden

--- data/glr.c  12 Sep 2005 05:50:44 -0000      1.120
+++ data/glr.c  15 Sep 2005 14:16:54 -0000      1.121
@@ -1674,6 +1674,11 @@ yyresolveValue (yySemanticOption* yyopti
              yybest = yyp;
              yymerge = yyfalse;
              break;
+           default:
+             /* This cannot happen so it is not worth a YYASSERT (false),
+                but some compilers complain if the default case is
+                omitted.  */
+             break;
            }
          yypp = &yyp->yynext;
        }