diff options
author | Christian Clason <c.clason@uni-graz.at> | 2022-02-17 14:56:03 +0100 |
---|---|---|
committer | Christian Clason <c.clason@uni-graz.at> | 2022-02-17 18:07:11 +0100 |
commit | e35a2d86fc2a8ed19aef00af9f35991385e833f1 (patch) | |
tree | 4a57eb61d3af92791f89428422f3f55623077bda /src | |
parent | f4e24f1eabfac439ac09d9646582f682c02cfe3f (diff) | |
download | rneovim-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.c | 4 |
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."); } |