Sorry, I sent this mail by accident before I had commented it.

Automake has some support for Lex and Yacc files intended for C++,
based on .yy, .y++ or even yxx.  It generates the corresponding .cc,
.c++ and .cxx, but does not apply the same transformation to headers.

There is also a couple of typos I think, I'm not sure:

| -    local ($yacc_suffix, $use_ylwrap, $c_suffix) = @_;
| +    local ($yacc_suffix, $use_ylwrap) = @_;

There is only one call to this function, and with two args, not
three.  Also, c_suffix was not used.

In order to be consistent with the lex-handling function, I used
c_suffix, not suffix.  Anyway, it is nicer if you introduce h_suffix.

| -    push (@suffixes, $yacc_suffix, $suffix);

Based on this in the Yacc code, I concluded that the lex code needed
to do the same, hence

| -    push (@suffixes, $lex_suffix);
| +    push (@suffixes, $lex_suffix, $c_suffix);

I have put those two variables in the v index, as some other
variables, but I'm not sure to understand the first two indexes:

@c Define an index of configure output variables.
@defcodeindex ov
@c Define an index of configure variables.
@defcodeindex cv

There are no uses of ov, and cv contains a lot of macro names:

~/src/automake % fgrep ovindex automake.texi                     nostromo 16:52
~/src/automake % fgrep cvindex automake.texi                     nostromo Err 1
@cvindex PACKAGE
@cvindex AM_INIT_AUTOMAKE
@cvindex AC_OUTPUT
@cvindex AC_CONFIG_HEADER
@cvindex AC_CONFIG_AUX_DIR
@cvindex AC_PATH_XTRA
@cvindex AC_CANONICAL_HOST
@cvindex AC_CHECK_TOOL
@cvindex AC_CANONICAL_SYSTEM
@cvindex AC_FUNC_ALLOCA

etc.

        Akim

Reply via email to