diff options
author | Mathias Fußenegger <mfussenegger@users.noreply.github.com> | 2024-07-27 22:30:14 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-07-27 22:30:14 +0200 |
commit | bdff50dee56ebf6de58d58315920abf2f8e262f7 (patch) | |
tree | aa1265a07bfe97b1283ee7ec808a679565fc0870 /runtime/lua/vim/_meta/options.lua | |
parent | aee4254b76206354d74d6fdd4d8e6b640279cedd (diff) | |
download | rneovim-bdff50dee56ebf6de58d58315920abf2f8e262f7.tar.gz rneovim-bdff50dee56ebf6de58d58315920abf2f8e262f7.tar.bz2 rneovim-bdff50dee56ebf6de58d58315920abf2f8e262f7.zip |
fix(lsp): revert text edit application order change (#29877)
Reverts https://github.com/neovim/neovim/pull/29212 and adds a few
additional test cases
From the spec
> All text edits ranges refer to positions in the document they are
> computed on. They therefore move a document from state S1 to S2 without
> describing any intermediate state. Text edits ranges must never overlap,
> that means no part of the original document must be manipulated by more
> than one edit. However, it is possible that multiple edits have the same
> start position: multiple inserts, or any number of inserts followed by a
> single remove or replace edit. If multiple inserts have the same
> position, the order in the array defines the order in which the inserted
> strings appear in the resulting text.
The previous fix seems wrong. The important part:
> If multiple inserts have the same position, the order in the array
> defines the order in which the inserted strings appear in the
> resulting text.
Emphasis on _appear in the resulting text_
Which means that in:
local edits1 = {
make_edit(0, 3, 0, 3, { 'World' }),
make_edit(0, 3, 0, 3, { 'Hello' }),
}
`World` must appear before `Hello` in the final text. That means the old
logic was correct, and the fix was wrong.
Diffstat (limited to 'runtime/lua/vim/_meta/options.lua')
0 files changed, 0 insertions, 0 deletions