GitHub/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim synced 2025-07-15 16:51:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(lsp): fix reverse sorting of same position text edits (#29212)

Browse Source 
Problem:
Text edits with the same position (both line and character) were being
reverse sorted prior to being applied which differs from the lsp spec

Solution:
Change the sort order for just the same position edits
This commit is contained in:
Al Colmenar
2024-06-07 02:54:43 -07:00
committed by GitHub
parent 6e45cd7f00
commit 2ce4a4d91e
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
runtime/lua/vim/lsp/util.lua
View File

@ -391,7 +391,7 @@ function M.apply_text_edits(text_edits, bufnr, offset_encoding)
return a.range.start.character > b.range.start.character
end
if a._index ~= b._index then
return a._index > b._index
return a._index < b._index
end
end)

12
test/functional/plugin/lsp_spec.lua
View File

@ -1789,9 +1789,9 @@ describe('LSP', function()
}
exec_lua('vim.lsp.util.apply_text_edits(...)', edits, 1, 'utf-16')
eq({
'',
'123',
'fooFbar',
'3',
'foo',
'12Fbar',
'123irst guy',
'baz line of text',
'The next line of text',
@ -1813,9 +1813,9 @@ describe('LSP', function()
}
exec_lua('vim.lsp.util.apply_text_edits(...)', edits, 1, 'utf-16')
eq({
'',
'123',
'fooFbar',
'3',
'foo',
'12Fbar',
'123irst guy',
'baz line of text',
'The next line of text',