Re: Lookahead correction for the C++ skeleton lalr1.cc

[Akim Demaille]

Hi Adrian,

Le 16 juil. 2019 à 11:32, Akim Demaille <address@hidden> a écrit :

> > > But stack.hh was really designed for symbols, not simple values such
> > > as state numbers. You may play with the appended protopatch if you
> > > feel like it, but otherwise I would just go for a plain std::vector for
> > > lac_stack_.
> > 
> > Personally, I would prefer std::vector
> 
> Yes, let's use it here.

I have received confirmation that your FSF papers were received, so I'll
install your patch (once the CI validates it).  I merged my changes into
yours, so I plan to push the following version of the patches.  Cheers!

Adrian Vogelsgesang (4):
  lalr1.cc: fix indentation of table declarations in the header
  lalr1.cc: reduce "scope"
  style: readability improvements to yacc.c
  lalr1.cc: add LAC support

Akim Demaille (2):
  tests: prepare LAC tests for more languages
  lalr1.cc: check LAC support

 data/skeletons/lalr1.cc | 237 ++++++++++++++++++++++++++++++++++++----
 data/skeletons/yacc.c   |  19 ++--
 doc/bison.texi          |   4 +-
 tests/conflicts.at      |  16 ++-
 tests/input.at          |  11 ++
 tests/regression.at     |  45 ++++----
 6 files changed, 276 insertions(+), 56 deletions(-)

-- 
2.22.0