Re: [PATCH 1/3] skeletons: use header guards.

[Akim Demaille]

Sorry I failed to send the introduction message (and actually lost
it).

There are two shortcomings.

(i) When using yacc.c, parser.c now includes parser.h instead of
duplicating its content.  glr.c and lalr1.cc already work like this.
Yet it means that parser.c must know where parser.h is in order to
#include it.  Currently all the implementation files (with yacc.c,
glr.c, lalr1.cc, glr.cc) include the basename of header file, so -I
should suffice to find it.  But instead of basename, I am
contemplating the use of code from coreutils to compute relative
paths, to handle cases such as --output=a/b/c/my-parser.c
--defines=a/d/my-header.h (but, as I reported elsewhere, I am stunned
by the impact this gnulib modules would have, comments most welcome:
http://lists.gnu.org/archive/html/bison-patches/2012-06/msg00023.html).

(ii) YYSTYPE and YYLTYPE are not renamed.  As already discussed, this
will be covered by some new %define api.prefix "foo" which will
deprecate %name-prefix.

I was clearly aiming at putting all this in 2.5.2, but, especially if
we decide that (ii) is important enough so that we wait for it to be
implemented, I start to believe that this should actually be 2.6 (and
master then would aim at 2.7).