aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorChristian Clason <c.clason@uni-graz.at>2022-02-17 14:56:03 +0100
committerChristian Clason <c.clason@uni-graz.at>2022-02-17 18:07:11 +0100
commite35a2d86fc2a8ed19aef00af9f35991385e833f1 (patch)
tree4a57eb61d3af92791f89428422f3f55623077bda /src
parentf4e24f1eabfac439ac09d9646582f682c02cfe3f (diff)
downloadrneovim-e35a2d86fc2a8ed19aef00af9f35991385e833f1.tar.gz
rneovim-e35a2d86fc2a8ed19aef00af9f35991385e833f1.tar.bz2
rneovim-e35a2d86fc2a8ed19aef00af9f35991385e833f1.zip
fix(api): allow empty list for cterm in nvim_set_hl
Problem: when accessing `nvim_set_hl` from Lua, empty tables are converted to empty lists, not dictionaries, resulting in an error for :lua vim.api.nvim_set_hl(0, "Comment", { cterm = {} }) Workaround: add an empty array as a special case when checking `dict->cterm.type` and just set `cterm_mask_provided`. (Proper solution: handle this in `gen_api_dispatch.lua`.)
Diffstat (limited to 'src')
-rw-r--r--src/nvim/highlight.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/nvim/highlight.c b/src/nvim/highlight.c
index e8a22ce6a8..8b998ff62e 100644
--- a/src/nvim/highlight.c
+++ b/src/nvim/highlight.c
@@ -876,6 +876,10 @@ HlAttrs dict2hlattrs(Dict(highlight) *dict, bool use_rgb, int *link_id, Error *e
CHECK_FLAG(cterm, cterm_mask, strikethrough, , HL_STRIKETHROUGH);
CHECK_FLAG(cterm, cterm_mask, nocombine, , HL_NOCOMBINE);
+ } else if (dict->cterm.type == kObjectTypeArray && dict->cterm.data.array.size == 0) {
+ // empty list from Lua API should clear all cterm attributes
+ // TODO(clason): handle via gen_api_dispatch
+ cterm_mask_provided = true;
} else if (HAS_KEY(dict->cterm)) {
api_set_error(err, kErrorTypeValidation, "'cterm' must be a Dictionary.");
}