Remove reference to LALR mode in readme

lalrpop / lalrpop

LR(1) parser generator for Rust

http://lalrpop.github.io/lalrpop/

Apache License 2.0

3.05k stars 292 forks source link

Remove reference to LALR mode in readme #586

Open Storyyeller opened 3 years ago

Storyyeller commented 3 years ago

Lalrpop used to support LALR generation as well as LR, but this was removed following the switch to lane tables. However, the readme claims that it is still possible to use LALR.

Storyyeller commented 3 years ago

Nevermind, it turns out that you can still use LALR by running lalrpop with LALRPOP_LANE_TABLE=disabled and putting [LALR] above your grammar() declaration. It's a shame this isn't better documented.

emdash commented 3 years ago

Maybe consider re-naming this issue to "Document how to use LALR mode", and then submit a PR to add this info to the readme?

nacaclanga commented 2 years ago

For some users, it might also be of interest to see, whether a grammar will also fullfill the LALR restrictions. In case this is easy to check, one could consider adding a restriction here, despite having no benefits over a full LR1 construction.