mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-09-10 21:38:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

api(extmarks): allow extrange past final newline

Browse Source
This commit is contained in:
Thomas Vigouroux
2020-09-08 14:14:15 +02:00
parent d4b4335fe3
commit cecc45efb1
2 changed files with 15 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/nvim/api/buffer.c
View File

@@ -1393,15 +1393,17 @@ Integer nvim_buf_set_extmark(Buffer buffer, Integer ns_id,
} }
if (col2 >= 0) { if (col2 >= 0) {
if (line2 >= 0) { if (line2 >= 0 && line2 < buf->b_ml.ml_line_count) {
len = STRLEN(ml_get_buf(buf, (linenr_T)line2+1, false)); len = STRLEN(ml_get_buf(buf, (linenr_T)line2 + 1, false));
} else if (line2 == buf->b_ml.ml_line_count) {
// We are trying to add an extmark past final newline
len = 0;
} else { } else {
// reuse len from before // reuse len from before
line2 = (int)line; line2 = (int)line;
} }
if (col2 > (Integer)len) { if (col2 > (Integer)len) {
api_set_error(err, kErrorTypeValidation, api_set_error(err, kErrorTypeValidation, "end_col value outside range");
"end_col value outside range");
goto error; goto error;
} }
} else if (line2 >= 0) { } else if (line2 >= 0) {

9
test/functional/api/extmark_spec.lua
View File

@@ -100,6 +100,15 @@ describe('API/extmarks', function()
ns2 = request('nvim_create_namespace', "my-fancy-plugin2") ns2 = request('nvim_create_namespace', "my-fancy-plugin2")
end) end)
it("can end extranges past final newline using end_col = 0", function()
set_extmark(ns, marks[1], 0, 0, {
end_col = 0,
end_line = 1
})
eq("end_col value outside range",
pcall_err(set_extmark, ns, marks[2], 0, 0, { end_col = 1, end_line = 1 }))
end)
it('adds, updates and deletes marks', function() it('adds, updates and deletes marks', function()
local rv = set_extmark(ns, marks[1], positions[1][1], positions[1][2]) local rv = set_extmark(ns, marks[1], positions[1][1], positions[1][2])
eq(marks[1], rv) eq(marks[1], rv)