Option to disable semicolon insertion

[9ary]

When completing certain keywords, rust-analyzer appends a semicolon which is not visible in the preview text. For example:

With return, it is sensible-enough to account for return type, but I still find it intrusive:

• this is actually inconsistent behavior
• it messes with my expectations and muscle memory, since the semicolon is not in the preview text and I am used to typing it manually
• with break and continue, it doesn't consider context; I may want to specify a label, but the semicolon is inserted even when inside of labeled loops, so I have to delete the semicolon in those cases

https://rust-analyzer.github.io/book/configuration.html#completion.addSemicolonToUnit exists, but from a cursory reading of the source code, this only seems to apply to function call snippets which I also have disabled.

[A4-Tacks]

Is adding a config to disable the semicolons for break, continue, and return what you expected?

[9ary]

@A4-Tacks yes, sorry if that was unclear. Other keywords as well, if any.

[A4-Tacks]

Other keywords as well, if any.

Exist "let" -> "let $1 = $0;", but it should not be included as the first cursor is not before or after the semicolon

[A4-Tacks]

@9ary It seems that this PR is not suitable. Do you have any ideas?

[9ary]

Not sure how I can help, other contributors seem more informed than I am and I would trust their opinion.