rneovim.git/test, branch fix_semantic_tokens_pull Neovim fork with Rahm's personal hacks. fix(semantic_tokens.lua): Fix nil tokens/data in semantic_tokens.lua 2023-01-19T20:42:40+00:00 Josh Rahm rahm@google.com 2023-01-19T19:37:12+00:00 409e6f9d7e690a398c4318a8eab7fd56a2eceda3 Some (poorly-implemented) LSPs can return an empty JSON object in LSP responses, which could cause tokens to be nil, which would eventually cause an error and a bad UI experience. This fix makes sure that the tokens variable is always set to a non-nil value. 
Some (poorly-implemented) LSPs can return an empty JSON object in
LSP responses, which could cause tokens to be nil, which would eventually
cause an error and a bad UI experience. This fix makes sure that the
tokens variable is always set to a non-nil value.
vim-patch:8.2.4655: cmdline completion popup menu positioned wrong (#21894) 2023-01-19T07:13:27+00:00 zeertzjq zeertzjq@outlook.com 2023-01-19T07:13:27+00:00 6e3890f4ce002ba308c09166b00e8ee86b6efa03 Problem: Command line completion popup menu positioned wrong when using a terminal window. Solution: Position the popup menu differently when editing the command line. (Yegappan Lakshmanan, closes vim/vim#10050, closes vim/vim#10035) https://github.com/vim/vim/commit/1104a6d0c2004d39e9b6cb8f804d12b628a69869 The test in the patch looks a bit hard to understand. Add a Lua test that is more straightforward. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com> 
Problem:    Command line completion popup menu positioned wrong when using a
            terminal window.
Solution:   Position the popup menu differently when editing the command line.
            (Yegappan Lakshmanan, closes vim/vim#10050, closes vim/vim#10035)

https://github.com/vim/vim/commit/1104a6d0c2004d39e9b6cb8f804d12b628a69869

The test in the patch looks a bit hard to understand.
Add a Lua test that is more straightforward.

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
 fix(statusline): don't leak memory with truncated click labels 2023-01-19T02:36:29+00:00 zeertzjq zeertzjq@outlook.com 2023-01-18T11:10:58+00:00 01ddc097474627f35cc34fb7015dd26505e2ddcc 






fix(statusline): don't leak memory with zero-width click labels
2023-01-19T02:36:29+00:00

zeertzjq
zeertzjq@outlook.com

2023-01-18T07:12:45+00:00

295a264b65bd072bf858496e0e5de83d5b1f5ffa

A zero-width click label isn't assigned to any click definition, so its
function name should be freed when filling click definitions.





A zero-width click label isn't assigned to any click definition, so its
function name should be freed when filling click definitions.







vim-patch:8.2.1698: cannot lock a variable in legacy Vim script like in Vim9 (#21883)
2023-01-19T00:03:13+00:00

zeertzjq
zeertzjq@outlook.com

2023-01-19T00:03:13+00:00

f8669e8a181abad98229d97439a9cafe8c32fc06

Problem:    Cannot lock a variable in legacy Vim script like in Vim9.
Solution:   Make ":lockvar 0" work.

https://github.com/vim/vim/commit/a187c43cfe8863d48b2159d695fedcb71f8525c1

Co-authored-by: Bram Moolenaar <Bram@vim.org>




Problem:    Cannot lock a variable in legacy Vim script like in Vim9.
Solution:   Make ":lockvar 0" work.

https://github.com/vim/vim/commit/a187c43cfe8863d48b2159d695fedcb71f8525c1

Co-authored-by: Bram Moolenaar <Bram@vim.org>






Merge pull request #21881 from bfredl/uniterror
2023-01-18T17:52:24+00:00

bfredl
bjorn.linse@gmail.com

2023-01-18T17:52:24+00:00

92a46727f767d16dfedeaf9a48459e0df1c7a082

fix(unittests): do not consider process crash to be a success




fix(unittests): do not consider process crash to be a success






fix(unittests): fix TUI broken test previously ignored
2023-01-18T13:56:55+00:00

bfredl
bjorn.linse@gmail.com

2023-01-18T12:59:40+00:00

6bfbb4db1d3708ce4ea69d29f3afe73def5a9f2a












fix(unittest): delete unused duplicated code
2023-01-18T13:56:55+00:00

bfredl
bjorn.linse@gmail.com

2023-01-18T11:12:13+00:00

847a1507aaf0877cda253ee4fdcb036fcd67816b

YAGNI. These were disabled 5 years ago in lint commit 29ed5b3a39abb84d6af602b2d0a7680d9dab381c





YAGNI. These were disabled 5 years ago in lint commit 29ed5b3a39abb84d6af602b2d0a7680d9dab381c







fix(unittests): do not consider process crash to be a success
2023-01-18T13:56:55+00:00

bfredl
bjorn.linse@gmail.com

2023-01-18T10:54:49+00:00

9fdb586592e24fbd42a42b1ea8a06fcb4a6dbf93

unittests relied on the exact setup of coredumps on CI to detect
process crashing, and otherwise completely discarded errors.

Dectect child process failure reliably using process status, so that
unittests actually work locally as well.





unittests relied on the exact setup of coredumps on CI to detect
process crashing, and otherwise completely discarded errors.

Dectect child process failure reliably using process status, so that
unittests actually work locally as well.







fix(statuscolumn): always fill click defs array (#21878)
2023-01-18T09:55:03+00:00

zeertzjq
zeertzjq@outlook.com

2023-01-18T09:55:03+00:00

36555730daaac8a9b88878a869c0cef6d5b701f0

Click definitions are always filled for tabline, statusline and winbar,
so they should also be always filled for statuscolumn, otherwise it will
leak memory.

Note: this doesn't actually change the existing code much, because of a
typo in the existing code.




Click definitions are always filled for tabline, statusline and winbar,
so they should also be always filled for statuscolumn, otherwise it will
leak memory.

Note: this doesn't actually change the existing code much, because of a
typo in the existing code.