cwbaker / lalr

LALR(1) parser for C++
MIT License
78 stars 11 forks source link

Fix wrong shift/reduce conflict detection #23

Closed mingodad closed 1 year ago

cwbaker commented 1 year ago

Thank you Domingo. I took a longer look at this and it turns out its a bug where the precedence and associativity set for < are overwritten by the precedence and associativity of template_test which names it as an implicit terminal in lalr.

Fixed by taking the precedence and associativity from which form of the implicit terminal they are set on. See #24.

cwbaker commented 1 year ago

Fixed by #24.