feat(extmarks): add strict option

The strict option, when set to false, allows placing extmarks on
invalid row and column values greater than the maximum buffer row
or line column respectively.

This allows for nvim_buf_set_extmark
to be a drop-in replacement for nvim_buf_set_highlight.
This commit is contained in:
Michael Lingelbach 2022-01-08 08:14:24 -08:00
parent b4fbb9dcf2
commit 11142f6ffe
2 changed files with 11 additions and 3 deletions

View File

@ -404,6 +404,10 @@ Array nvim_buf_get_extmarks(Buffer buffer, Integer ns_id, Object start, Object e
/// for left). Defaults to false.
/// - priority: a priority value for the highlight group. For
/// example treesitter highlighting uses a value of 100.
/// - strict: boolean that indicates extmark should be placed
/// even if the line or column value is past the end of the
/// buffer or end of the line respectively. Defaults to true.
///
/// @param[out] err Error details, if any
/// @return Id of the created/updated extmark
Integer nvim_buf_set_extmark(Buffer buffer, Integer ns_id, Integer line, Integer col,
@ -596,7 +600,10 @@ Integer nvim_buf_set_extmark(Buffer buffer, Integer ns_id, Integer line, Integer
bool ephemeral = false;
OPTION_TO_BOOL(ephemeral, ephemeral, false);
if (line < 0 || line > buf->b_ml.ml_line_count) {
bool strict = true;
OPTION_TO_BOOL(strict, strict, true);
if (line < 0 || (line > buf->b_ml.ml_line_count && strict)) {
api_set_error(err, kErrorTypeValidation, "line value outside range");
goto error;
} else if (line < buf->b_ml.ml_line_count) {
@ -605,7 +612,7 @@ Integer nvim_buf_set_extmark(Buffer buffer, Integer ns_id, Integer line, Integer
if (col == -1) {
col = (Integer)len;
} else if (col < -1 || col > (Integer)len) {
} else if (col < -1 || (col > (Integer)len && strict)) {
api_set_error(err, kErrorTypeValidation, "col value outside range");
goto error;
}
@ -620,7 +627,7 @@ Integer nvim_buf_set_extmark(Buffer buffer, Integer ns_id, Integer line, Integer
// reuse len from before
line2 = (int)line;
}
if (col2 > (Integer)len) {
if (col2 > (Integer)len && strict) {
api_set_error(err, kErrorTypeValidation, "end_col value outside range");
goto error;
}

View File

@ -21,6 +21,7 @@ return {
"virt_lines";
"virt_lines_above";
"virt_lines_leftcol";
"strict";
};
keymap = {
"noremap";