Multi cursor postfix snippets only work correctly for the first cursor

[tjallingt]

rust-analyzer version: bc08b8e 2022-03-28 stable

rustc version: 1.58.1 (db9d1b20b 2022-01-20)

relevant settings: N/A

match result {
    Some(_) => {
        assert!(wip.is_empty());
        /* ... */
    },
    None => {
        assert!(wip.is_empty());
        /* ... */
    }
};

In code above I want to quicky invert the assert conditions, so I select the first wip.is_empty() and select the next occurence with Ctrl+D. Then I type .not at the end and press enter.

The result is

match result {
    Some(_) => {
        assert!(!wip.is_empty());
        /* ... */
    },
    None => {
        assert!(wip.is_empty().!wip.is_empty());
        /* ... */
    }
};

(This may be an issue with VS Code, but I don't know how to tell. If so I can close this and report it to VS Code)

[Veykril]

This is blocked on the LSP not providing us with multiple cursors, and it doesn't seem to be on the roadmap currently: microsoft/language-server-protocol#22

But maybe we are able to put this in as an LSP extension.

[flodiebold]

I don't know, that seems like a VSCode bug to me. There are probably multiple ways one could think this should work, but the actual result is not one of them.

(And I somewhat agree with the points made in the LSP issue that this doesn't seem like something LSP should be concerned with either.)

[lnicola]

FWIW, https://helix-editor.com/news/release-22-12-highlights/ shows this already working, in a way.