diff --git a/src/nvim/syntax.c b/src/nvim/syntax.c index 70a2efb588..68f3329ce7 100644 --- a/src/nvim/syntax.c +++ b/src/nvim/syntax.c @@ -7762,10 +7762,18 @@ static void highlight_list_two(int cnt, int attr) * Function given to ExpandGeneric() to obtain the list of group names. * Also used for synIDattr() function. */ -const char *get_highlight_name(expand_T *const xp, const int idx) +const char *get_highlight_name(expand_T *const xp, int idx) FUNC_ATTR_WARN_UNUSED_RESULT { - // TODO(justinmk): 'xp' is unused + if (idx < 0) { + return NULL; + } + + // Items are never removed from the table, skip the ones that were cleared. + while (idx < highlight_ga.ga_len && HL_TABLE()[idx].sg_cleared) { + idx++; + } + if (idx == highlight_ga.ga_len && include_none != 0) { return "none"; } else if (idx == highlight_ga.ga_len + include_none @@ -7777,20 +7785,10 @@ const char *get_highlight_name(expand_T *const xp, const int idx) } else if (idx == highlight_ga.ga_len + include_none + include_default + 1 && include_link != 0) { return "clear"; - } else if (idx < 0) { + } else if (idx >= highlight_ga.ga_len) { return NULL; } - - // Items are never removed from the table, skip the ones that were cleared. - int current_idx = idx; - while (current_idx < highlight_ga.ga_len - && HL_TABLE()[current_idx].sg_cleared) { - current_idx++; - } - if (current_idx >= highlight_ga.ga_len) { - return NULL; - } - return (const char *)HL_TABLE()[current_idx].sg_name; + return (const char *)HL_TABLE()[idx].sg_name; } color_name_table_T color_name_table[] = { diff --git a/src/nvim/testdir/test_cmdline.vim b/src/nvim/testdir/test_cmdline.vim index 1bf5930eb9..94330fe67b 100644 --- a/src/nvim/testdir/test_cmdline.vim +++ b/src/nvim/testdir/test_cmdline.vim @@ -46,8 +46,34 @@ func Test_map_completion() call assert_equal('"map ', getreg(':')) endfunc +func Test_match_completion() + if !has('cmdline_compl') + return + endif + hi Aardig ctermfg=green + call feedkeys(":match \\\"\", 'xt') + call assert_equal('"match Aardig', getreg(':')) + call feedkeys(":match \\\"\", 'xt') + call assert_equal('"match none', getreg(':')) +endfunc + +func Test_highlight_completion() + if !has('cmdline_compl') + return + endif + hi Aardig ctermfg=green + call feedkeys(":hi \\\"\", 'xt') + call assert_equal('"hi Aardig', getreg(':')) + call feedkeys(":hi li\\\"\", 'xt') + call assert_equal('"hi link', getreg(':')) + call feedkeys(":hi d\\\"\", 'xt') + call assert_equal('"hi default', getreg(':')) + call feedkeys(":hi c\\\"\", 'xt') + call assert_equal('"hi clear', getreg(':')) +endfunc + func Test_expr_completion() - if !(has('cmdline_compl') && has('eval')) + if !has('cmdline_compl') return endif for cmd in [ @@ -286,17 +312,17 @@ func Test_paste_in_cmdline() endfunc func Test_remove_char_in_cmdline() - call feedkeys(":abc def\\\\"\", 'tx') - call assert_equal('"abc ef', @:) + call feedkeys(":abc def\\\\"\", 'tx') + call assert_equal('"abc ef', @:) - call feedkeys(":abc def\\\\"\", 'tx') - call assert_equal('"abcdef', @:) + call feedkeys(":abc def\\\\"\", 'tx') + call assert_equal('"abcdef', @:) - call feedkeys(":abc def ghi\\\\"\", 'tx') - call assert_equal('"abc ghi', @:) + call feedkeys(":abc def ghi\\\\"\", 'tx') + call assert_equal('"abc ghi', @:) - call feedkeys(":abc def\\\\"\", 'tx') - call assert_equal('"def', @:) + call feedkeys(":abc def\\\\"\", 'tx') + call assert_equal('"def', @:) endfunc func Test_illegal_address1()