Re: push-pull syntax varies between Bison versions

[John W. Eaton]

On 02/28/2013 01:53 PM, Rik wrote:
> 2/28/13
>
> John,
>
> The new directive for a push-pull parser is currently
>
> %define api.push-pull both
>
> which is appropriate for bison versions>= 2.5.
>
> The original syntax that they introduced in 2.4 was
>
> %define api.push_pull "both"
>
> which has underscores and quoting.  It looks like an autoconf test is in
> order to verify which syntax the host is using.

OK, well this is a bit of a work in progress...

After checking in my last change, I realized that I also checked 
USE_PUSH_PARSER in the sources but set OCTAVE_USE_PUSH_PARSER in 
configure.ac, so wasn't even testing the push parser when using 
--enable-push-parser.  Oops.  I'm working on that now.

I guess something like

  #ifdef BISON_NEEDS_QUOTED_PARSER_TYPES
  # define PARSER_TYPES "both"
  #else
  # define PARSER_TYPES both
  #endif

  ...

  %define api.push-pull PARSER_TYPES

won't work, so this means generating oct-parse.yy from oct-parse.in.yy, 
right?  Ugh.

jwe