Re: FYI: lalr1.cc: Move tokens and YYSTYPE into the class

[Paul Eggert]

I found a minor glitch with that patch: xgettext complains about the
unmatched ' in lalr1.cc:

  data/lalr1.cc:33: warning: unterminated character constant

I installed the patch enclosed below.
(Don't you just love the quoting rules here?  :-)


Also, could you please explain the new FIXME comment here?  Sorry, I
don't quite follow it.  What macros used to be there, but are missing now?


2005-10-02  Paul Eggert  <address@hidden>

        * data/lalr1.cc: Don't put an unmatched ' in a dnl comment, as
        this confuses xgettext.

--- data/lalr1.cc       2 Oct 2005 17:44:49 -0000       1.105
+++ data/lalr1.cc       2 Oct 2005 20:37:28 -0000
@@ -30,7 +30,7 @@ m4_if(b4_defines_flag, 0, [],
 b4_copyright([C++ Skeleton parser for LALR(1) parsing with Bison],
              [2002, 2003, 2004, 2005])
 dnl FIXME: This is wrong, we want computed header guards.
-dnl FIXME: I don\'t know why the macros are missing now. :(
+dnl FIXME: I do not know why the macros are missing now. :(
 [
 #ifndef PARSER_HEADER_H
 # define PARSER_HEADER_H