IntenseFinance/neovim
3
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-02-25 18:55:25 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix(lsp): fix nil-index behavior for UTF-8 in _str_*index_enc methods (#16731)

Browse Source 
Previously, the `_str_utfindex_enc` and `_str_byteindex_enc` helper functions would return `nil` when `offset_encoding == "utf-8"` and `index == nil`. Clearly, this doesn't reflect the expected behavior of the functions they're wrapping which would return the length of the line in this case. This should fix behavior with servers that use UTF-8 `offset_encoding` when applying text edits, formatting a range, and doing range code actions (though this isn't tested currently).
This commit is contained in:
Rishikesh Vaishnav 2021-12-20 08:54:05 -08:00 committed by GitHub
parent 63b21ab30d
commit 9625832372
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -97,15 +97,17 @@ end
---@param encoding string utf-8|utf-16|utf-32|nil defaults to utf-16 ---@param encoding string utf-8|utf-16|utf-32|nil defaults to utf-16
---@return number `encoding` index of `index` in `line` ---@return number `encoding` index of `index` in `line`
function M._str_utfindex_enc(line, index, encoding) function M._str_utfindex_enc(line, index, encoding)
if encoding ~= 'utf-8' then if not encoding then encoding = 'utf-16' end
local col32, col16 = vim.str_utfindex(line, index) if encoding == 'utf-8' then
if encoding == 'utf-32' then if index then return index else return #line end
return col32 elseif encoding == 'utf-16' then
else local _, col16 = vim.str_utfindex(line, index)
return col16 return col16
end elseif encoding == 'utf-32' then
local col32, _ = vim.str_utfindex(line, index)
return col32
else else
return index error("Invalid encoding: " .. vim.inspect(encoding))
end end
end end
@ -117,10 +119,15 @@ end
---@param encoding string utf-8|utf-16|utf-32|nil defaults to utf-16 ---@param encoding string utf-8|utf-16|utf-32|nil defaults to utf-16
---@return number byte (utf-8) index of `encoding` index `index` in `line` ---@return number byte (utf-8) index of `encoding` index `index` in `line`
function M._str_byteindex_enc(line, index, encoding) function M._str_byteindex_enc(line, index, encoding)
if encoding ~= 'utf-8' then if not encoding then encoding = 'utf-16' end
return vim.str_byteindex(line, index, not encoding or encoding ~= 'utf-32') if encoding == 'utf-8' then
if index then return index else return #line end
elseif encoding == 'utf-16' then
return vim.str_byteindex(line, index, true)
elseif encoding == 'utf-32' then
return vim.str_byteindex(line, index)
else else
return index error("Invalid encoding: " .. vim.inspect(encoding))
end end
end end