search

AzureMarker / intellij-lalrpop

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

refactoring imported variable names doubles use and semi colon termination #48

Closed ghost closed 2 years ago

ghost commented 2 years ago

If you have something in your lalrpop grammar file at the top like:

use crate::parser::ast::{SomeAst, SomeOtherAst};

And in your parser::ast module you right click SomeAst and choose Refactor > Rename... to AnotherAst you then get invalid syntax:

use use crate::parser::ast::{AnotherAst, SomeOtherAst};;

Simple to manually correct, but appears to be a consistent issue so thought I'd report it in case it's easy to resolve!

Plugin version: 0.2.6 IntelliJ version: 2021.2.1 (Build: #IC-212.5080.55)

AzureMarker commented 2 years ago

I couldn't reproduce this due to another issue (https://github.com/intellij-rust/intellij-rust/issues/7655) but I'm adding a possible fix in the next release. Please let me know if it fixes the issue for you.

Edit: Released v0.2.7, re-open if it isn't fixed.