Closed MiSawa closed 2 years ago
Thanks for reporting this and the PR.
lalrpop compatibility has always been a goal since the early days, but I never got around using lexgen with lalrpop as I originally planned, instead I wrote my own parsers.
Btw, if you're interested in implementing a lexgen-lalrpop example or a test I would be happy to merge that as well :-)
Opened PR that adds an example! :) https://github.com/osa1/lexgen/pull/46
Hi, I'm wondering if this change can be published so we don't need to specify dependency via lexgen = { git = "..." }
:)
Hi, I'm wondering if this change can be published so we don't need to specify dependency via lexgen = { git = "..." } :)
Done :)
Awesome, thank you!!
Thank you for the amazing lexer!
The generated lexer seems to be 99% compatible with what lalrpop takes. The only incompatibility is the lack of
Default
implementation forlexgen_util::Loc
, which lalrpop uses to initialize the location. If there's no reason not to implement it, it'd be great to have it implemented so that the following just works (withtype Location = lexgen_util::Loc
andtype Error = lexgen_util::LexerError<MyError>
defined in theextern
section of the.lalrpop
file).