aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* vim-patch:8.2.2249: termcodes test is flaky when used over sshzeertzjq2023-05-06
| | | | | | | | Problem: Termcodes test is flaky when used over ssh with X forwarding. Solution: Set 'mousetime' to a larger value. (Dominique Pellé, closes vim/vim#7576, closes vim/vim#7563) https://github.com/vim/vim/commit/2a5c61a0196d2d67313f5b3189974211e6f33562
* vim-patch:9.0.0904: various comment and indent flaws (#23498)zeertzjq2023-05-06
| | | | | | | | | | | Problem: Various comment and indent flaws. Solution: Improve comments and indenting. https://github.com/vim/vim/commit/88456cd3c49a3dd1fda17cf350daa9b8216b1aa6 Omit test_function_lists.vim change as that file is likely not applicable to Nvim due to the existence of Nvim-only functions. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* vim-patch:8.2.1746: Vim9: cannot use "fina" for "finally" (#23497)zeertzjq2023-05-06
| | | | | | | | | | Problem: Vim9: Cannot use "fina" for "finally". (Naruhiko Nishino) Solution: Specifically check for "fina". (closes vim/vim#7020) https://github.com/vim/vim/commit/373863ed48c02b5df52574aa7d50aeecb1037d40 :final is N/A. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23488 from zeertzjq/vim-8.2.2094zeertzjq2023-05-06
|\ | | | | vim-patch:8.2.{2094,2141},9.0.1508
| * vim-patch:9.0.1508: catch does not work when lines are joined with a newlinezeertzjq2023-05-06
| | | | | | | | | | | | | | Problem: Catch does not work when lines are joined with a newline. Solution: Set "nextcmd" appropriately. (closes vim/vim#12348) https://github.com/vim/vim/commit/f2588b6fc90ba85b333ee8f747e8d1ebbc7e6300
| * vim-patch:8.2.2141: a user command with try/catch may not catch an ↵zeertzjq2023-05-06
| | | | | | | | | | | | | | | | | | | | | | expression error Problem: A user command with try/catch may not catch an expression error. Solution: When an expression fails check for following "|". (closes vim/vim#7469) https://github.com/vim/vim/commit/8143a53c533bc7776c57e5db063d185bdd5750f3 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.2094: when an expression fails getting next command may be wrongzeertzjq2023-05-06
|/ | | | | | | | | Problem: When an expression fails getting the next command may be wrong. Solution: Do not check for a next command after :eval fails. (closes vim/vim#7415) https://github.com/vim/vim/commit/d0fe620cbbf5f5e00446efa89893036265c5c302 Co-authored-by: Bram Moolenaar <Bram@vim.org>
* fix(api): don't change title when setting buffer in a window (#23492)zeertzjq2023-05-06
|
* fix(health): replace healthFoo with DiagnosticFoo (#23475)Christian Clason2023-05-05
| | | | | | This replaces the custom `health{Error,Warning,Success}` highlight groups with `Diagnostic{Error,Warning,Ok}`, which are defined by default. Removes the link for `healthHelp`, which was no longer actually used after #20879.
* Merge pull request #23487 from zeertzjq/vim-8.2.1953zeertzjq2023-05-05
|\ | | | | vim-patch:8.2.1953,9.0.{0213,0404,0543,0846,0854,1507}: assert fixes
| * vim-patch:9.0.1507: assert message is confusing with boolean resultzeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | | | | | Problem: Assert message is confusing with boolean result. assert_inrange() replaces message instead of adding it. Solution: Don't put quotes around expected boolean value. Append message for assert_inrange(). (closes vim/vim#12342, closes vim/vim#12341) https://github.com/vim/vim/commit/53f5e51628b56ef9171671cd6e9970374036a084 Move assert_type_T to testing.c and remove ASSERT_INRANGE.
| * vim-patch:9.0.0854: no proper test for what 9.0.0846 fixeszeertzjq2023-05-05
| | | | | | | | | | | | | | | | Problem: No proper test for what 9.0.0846 fixes. Solution: Run test in a terminal so that the hit-enter prompt can show up. (closes vim/vim#11523) https://github.com/vim/vim/commit/7265851b2b4e5a63c0a02a9057dee237502ee557
| * vim-patch:9.0.0846: using assert_fails() may cause hit-enter promptzeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Using assert_fails() may cause hit-enter prompt. Solution: Set no_wait_return. (closes vim/vim#11522) https://github.com/vim/vim/commit/f220643c260d55d21a841a3c4032daadc41bc50b Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:9.0.0543: insufficient testing for assert and test functionszeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insufficient testing for assert and test functions. Solution: Add a few more tests. (Yegappan Lakshmanan, closes vim/vim#11190) https://github.com/vim/vim/commit/e24b5e0b0f5ab015215ef2761baa98ccb1ba8606 Cherry-pick E1219 from patch 8.2.3229. Cherry-pick test_assert.vim change from patch 9.0.0491. Cherry-pick the whole Test_refcount() function and skip it. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
| * vim-patch:9.0.0404: crash when passing invalid arguments to assert_fails()zeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Crash when passing invalid arguments to assert_fails(). Solution: Check for NULL string. https://github.com/vim/vim/commit/1540d334a04d874c2aa9d26b82dbbcd4bc5a78de Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:9.0.0213: using freed memory with error in assert argumentzeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Using freed memory with error in assert argument. Solution: Make a copy of the error. https://github.com/vim/vim/commit/249e1b903a9c0460d618f6dcc59aeb8c03b24b20 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.1953: Vim9: extra "unknown" error after other errorzeertzjq2023-05-05
|/ | | | | | | | | | Problem: Vim9: extra "unknown" error after other error. Solution: Restore did_emsg count after EXEC instruction. (closes vim/vim#7254) Improve error message from assert_fails() https://github.com/vim/vim/commit/631e8f93458b102091d54c502f489c03e454d4da Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23482 from zeertzjq/do-cmdline-verbosezeertzjq2023-05-05
|\ | | | | fix(excmd): append original command to error message
| * test: add tests for executing commands with backwards rangezeertzjq2023-05-05
| |
| * fix(excmd): append original command to error messagezeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | | | Revert the change to do_cmdline_cmd() from #5226. This function is used in many places, so making it different from Vim leads to small differences from Vim in the behavior of some functions like execute() and assert_fails(). If DOCMD_VERBOSE really needs to be removed somewhere, a do_cmdline() call without DOCMD_VERBOSE is also shorter than a do_cmdline() call with DOCMD_VERBOSE.
* | perf(lsp): load buffer contents once when processing semantic tokens ↵jdrouhard2023-05-05
|/ | | | | | | | | | | | | | | | | | | | | | | | | | responses (#23484) perf(lsp): load buffer contents once when processing semantic token responses Using _get_line_byte_from_position() for each token's boundaries was a pretty huge bottleneck, since that function would load individual buffer lines via nvim_buf_get_lines() (plus a lot of extra overhead). So each token caused two calls to nvim_buf_get_lines() (once for the start position, and once for the end position). For semantic tokens, we only attach to buffers that have already been loaded, so we can safely just get all the lines for the entire buffer at once, and lift the rest of the _get_line_byte_from_position() implementation directly while bypassing the part that loads the buffer line. While I was looking at get_lines (used by _get_line_byte_from_position), I noticed that we were checking for non-file URIs before we even looked to see if we already had the buffer loaded. Moving the buffer-loaded check to be the first thing done in get_lines() more than halved the average time spent transforming the token list into highlight ranges vs when it was still using _get_line_byte_from_position. I ended up improving that loop more by not using get_lines, but figured the performance improvement it provided was worth leaving in.
* Merge pull request #23483 from zeertzjq/vim-8.2.3135zeertzjq2023-05-05
|\ | | | | vim-patch:8.2.{3135,4890,4892},9.0.0250: error message improvements
| * vim-patch:9.0.0250: slightly inconsistent error messageszeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Slightly inconsistent error messages. Solution: Make it "Using a Float". (closes vim/vim#10959) https://github.com/vim/vim/commit/dde77a7c4d72622284dc5fb72557bde42c314fa6 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.4892: test failures because of changed error messageszeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Test failures because of changed error messages. Solution: Adjust the exptected error messages. https://github.com/vim/vim/commit/ec892234788d37bfed47d8bf82dd4cae8d335ad9 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.4890: inconsistent capitalization in error messageszeertzjq2023-05-05
| | | | | | | | | | | | | | | | | | Problem: Inconsistent capitalization in error messages. Solution: Make capitalization consistent. (Doug Kearns) https://github.com/vim/vim/commit/cf030578b26460643dca4a40e7f2e3bc19c749aa Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * refactor: using a different error number for 'mousescroll'zeertzjq2023-05-05
| | | | | | | | Because E548 is linked to 'guicursor' in help.
| * vim-patch:8.2.3135: Vim9: builtin function arguments not checked at compile timezeertzjq2023-05-05
|/ | | | | | | | | Problem: Vim9: builtin function arguments not checked at compile time. Solution: Add more type checks. (Yegappan Lakshmanan, closes vim/vim#8539) https://github.com/vim/vim/commit/5b73992d8f82be7ac4b6f46c17f53ffb9640e5fa Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
* vim-patch:8.2.4397: crash when using many composing characters in error ↵zeertzjq2023-05-05
| | | | | | | | | | message (#23481) Problem: Crash when using many composing characters in error message. Solution: Use mb_cptr2char_adv() instead of mb_ptr2char_adv(). https://github.com/vim/vim/commit/34f8117dec685ace52cd9e578e2729db278163fc Co-authored-by: Bram Moolenaar <Bram@vim.org>
* vim-patch:8.2.3221: Vim9: argument types are not checked at compile time ↵zeertzjq2023-05-05
| | | | | | | | | | | | (#23480) Problem: Vim9: argument types are not checked at compile time. Solution: Add several more type checks. (Yegappan Lakshmanan, closes vim/vim#8632) https://github.com/vim/vim/commit/a764e73d4ffc5d046807c757eaacb9b0a5408152 Cherry-pick test_assert.vim change from patch 8.2.3229. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
* vim-patch:8.2.1697: inconsistent capitalization of error messages (#23476)zeertzjq2023-05-04
| | | | | | | | | | Problem: Inconsistent capitalization of error messages. Solution: Always start with a capital. https://github.com/vim/vim/commit/7707228aace9aff16434edf5377a354c6ad07316 Most of these errors are Vim9 script only. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* vim-patch:9.0.0601: too much indent (#23472)zeertzjq2023-05-04
| | | | | | | | | Problem: Too much indent. Solution: Return out early from a funtion. (Yegappan Lakshmanan, close vim/vim#11238) https://github.com/vim/vim/commit/368aa6908862874fdb901c534ee99033ac977e4a Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
* Merge pull request #23470 from zeertzjq/vim-8.2.2344zeertzjq2023-05-04
|\ | | | | vim-patch:8.2.{1461,1462,1466,2344,2607,2756}
| * vim-patch:8.2.2756: Vim9: blob index and slice not implemented yetzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: Vim9: blob index and slice not implemented yet. Solution: Implement blob index and slice. https://github.com/vim/vim/commit/cfc3023cb6ce5aaec13f49bc4b821feb05e3fb03 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.2607: strcharpart() cannot include composing characterszeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: strcharpart() cannot include composing characters. Solution: Add the {skipcc} argument. https://github.com/vim/vim/commit/02b4d9b18a03549b68e364e428392b7a62766c74 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.2344: using inclusive index for slice is not always desiredzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Using inclusive index for slice is not always desired. Solution: Add the slice() method, which has an exclusive index. (closes vim/vim#7408) https://github.com/vim/vim/commit/6601b62943a19d4f8818c3638440663d67a17b6a Cherry-pick a line in docs added later. Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.1466: Vim9: cannot index or slice a variable with type "any"zeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | | | | | Problem: Vim9: cannot index or slice a variable with type "any". Solution: Add runtime index and slice. https://github.com/vim/vim/commit/cc673e746ab98566556ff964d7a76f2fb46d7f84 Missing changes from the last PR. Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.1462: Vim9: string slice not supported yetzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: Vim9: string slice not supported yet. Solution: Add support for string slicing. https://github.com/vim/vim/commit/11107bab7ead9124f46a7ddf6aa3bb66b43a8246 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.1461: Vim9: string indexes are counted in byteszeertzjq2023-05-04
|/ | | | | | | | | Problem: Vim9: string indexes are counted in bytes. Solution: Use character indexes. (closes vim/vim#6574) https://github.com/vim/vim/commit/e3c37d8ebf9dbbf210fde4a5fb28eb1f2a492a34 Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23468 from zeertzjq/vim-8.2.1466zeertzjq2023-05-04
|\ | | | | vim-patch:8.2.{1466,3336}
| * vim-patch:8.2.3336: behavior of negative index in list change changedzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | | | Problem: Behavior of negative index in list change changed. (Naruhiko Nishino) Solution: Only change it for Vim9 script. (closes vim/vim#8749) https://github.com/vim/vim/commit/92f05f21afdb8a43581554a252cb2fc050f9e03b Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.1466: Vim9: cannot index or slice a variable with type "any"zeertzjq2023-05-04
|/ | | | | | | | | | | | | Problem: Vim9: cannot index or slice a variable with type "any". Solution: Add runtime index and slice. https://github.com/vim/vim/commit/cc673e746ab98566556ff964d7a76f2fb46d7f84 Omit E1024 and E1062: Vim9 script only. Omit string_slice() and char_idx2byte(): Vim9 script only. Remove the first tv_is_luafunc() check because it always returns false. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23467 from zeertzjq/vim-8.2.0987zeertzjq2023-05-04
|\ | | | | vim-patch:8.2.{0987,1463}
| * vim-patch:8.2.1463: Vim9: list slice not supported yetzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: Vim9: list slice not supported yet. Solution: Add support for list slicing. https://github.com/vim/vim/commit/ed5918771fcf9877d8445e74c62ab1ce6b8e28c1 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:8.2.0987: Vim9: cannot assign to [var; var]zeertzjq2023-05-04
|/ | | | | | | | | Problem: Vim9: cannot assign to [var; var]. Solution: Assign rest of items to a list. https://github.com/vim/vim/commit/9af78769eeae0318e07aa8b6af4d6e2244481ca7 Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23466 from zeertzjq/vim-9.0.0138zeertzjq2023-05-04
|\ | | | | vim-patch:9.0.{0138,0240}: two spell fixes
| * vim-patch:9.0.0240: crash when using ":mkspell" with an empty .dic filezeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: Crash when using ":mkspell" with an empty .dic file. Solution: Check for an empty word tree. https://github.com/vim/vim/commit/6669de1b235843968e88844ca6d3c8dec4b01a9e Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:9.0.0138: not enough characters accepted for 'spellfile'zeertzjq2023-05-04
|/ | | | | | | | | | | Problem: Not enough characters accepted for 'spellfile'. Solution: Add vim_is_fname_char() and use it for 'spellfile'. https://github.com/vim/vim/commit/bc49c5f48f89c2d6f4d88ee77f44a11d68293be3 Cherry-pick related doc update from Vim runtime. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge pull request #23464 from zeertzjq/vim-9.0.0609zeertzjq2023-05-04
|\ | | | | vim-patch:9.0.{0609,0612}
| * vim-patch:9.0.0612: blockedit test passes with wrong resultzeertzjq2023-05-04
| | | | | | | | | | | | | | | | | | Problem: Blockedit test passes with wrong result. Solution: Add a "vim9script" line to make indenting work. https://github.com/vim/vim/commit/47da934844afec7b04d94e0d1f8cf0a86e1b9bea Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:9.0.0609: blockedit test fails because of wrong indentzeertzjq2023-05-04
|/ | | | | | | | | Problem: Blockedit test fails because of wrong indent. Solution: Adjust the expected text temporarily https://github.com/vim/vim/commit/66000ff9af8e3de93825ce7baa0c43727465eca5 Co-authored-by: Bram Moolenaar <Bram@vim.org>