search

AzureMarker / intellij-lalrpop

Jetbrains plugin for the LALRPOP parser-generator
MIT License
16 stars 2 forks source link

More inspections #22

Closed dnbln closed 3 years ago

dnbln commented 3 years ago

This PR adds inspections for the first 3 ideas mentioned in #18 + 3 extra.

AzureMarker commented 3 years ago

@dblanovschi Let me know when you're ready for this PR to be merged. I'll probably make a release right afterwards.

dnbln commented 3 years ago

With the last commit, there is only https://github.com/Mcat12/intellij-lalrpop/pull/22#discussion_r536823425 left. I think that'd be worth looking into, but some other time. ~There is nothing else I'd like to add here.~ Edit: I just found out that precedence annotations were merged onto lalrpop, so I'd like to add an inspection to validate them (the rules described in "Prevalidation" here at least)