Do not insert matching }

Although I’m very happy with intellij-rust, one feature annoys me.

Intellij-rust inserts {} when I type {. It is annoying, because most of the time I need to erase that matching brace.

E. g. I have a code like this:

match foo {
  Bar(x) <CURSOR HERE> if x => 1,
}

I want to type => { at cursor position to convert code to:

match foo {
  Bar(x) => { if x { 1 } else { 2 } },
}

However, Intellij-rust inserts:

match foo {
  Bar(x) {} if x => 1,
}

so I need to immediately erase }.

Also, intellij-rust is very good at inserting matching } when I press Enter, e. g. in this case:

match foo {
  Bar(x) {<CURSOR HERE if x => 1,
}

So matching } right after { is almost never needed.

[alygin]

I never paid much attention to that, but for Java, IDEA only inserts matching }s in some predefined contexts. Unfortunately, it sometimes doesn’t insert them when I would expect it to do, but at least it doesn’t annoy me when I do some nontrivial refactoring. Kotlin only analyzes the following symbol, CLion seems to not care about it at all and just throws } everywhere.

What do you think about such approach: try to insert a } and if it causes a syntax error nearby, remove it? I never looked into this part of the plugin, so not sure if there could be a reliable implementation.

[matklad]

This option also stops inserting matching ‘(’ and ‘[’ which works well. But thanks anyway.

Yeah, I think we need just to be more smart and do the same thing that Java does. I am in general against adding more options, I would prefer IDE to just do the right thing.

By the way, @la10736 there’s a nice option “surround selection on typing quote or brace”, which is a little bit nicer to use than surrounders.