aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | 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>
* | fix(editorconfig): add missing root validation (#23462)Bogdan Grigoruță2023-05-03
| |
* | Merge pull request #23428 from luukvbaal/gridclearbfredl2023-05-03
|\ \ | | | | | | fix(tui): grid_clear properly clears the screen
| * | fix(tui): grid_clear properly clears the screenLuuk van Baal2023-05-03
| | | | | | | | | | | | | | | | | | | | | Problem: When setting a shell size smaller than the containing terminal window through `:winsize` or `:set lines/columns` the screen is not properly cleared. Solution: Clear the tui dimensions rather than the grid dimensions.
* | | fix(mouse): fix popup menu position check with winbar (#23456)zeertzjq2023-05-03
| | |
* | | vim-patch:9.0.1028: mouse shape test is flaky, especially on Mac OS (#23455)zeertzjq2023-05-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Mouse shape test is flaky, especially on Mac OS. Solution: Instead of starting all timers at the same time, start the next one in the callback of the previous one. (Yee Cheng Chin, closes vim/vim#11673) Also use "bwipe!" instead of "close!" to avoid swap files remaining. https://github.com/vim/vim/commit/1881abfc34a61e7fb90bbcf8a5b8550405d55613 Cherry-pick test_normal.vim changes from patch 8.2.3795. Co-authored-by: Yee Cheng Chin <ychin.git@gmail.com>
* | | vim-patch:9.0.0662: concealed characters do not work correctly (#23454)zeertzjq2023-05-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Concealed characters do not work correctly. Solution: Subtract boguscols instead of adding them. (closes vim/vim#11273) https://github.com/vim/vim/commit/75008661821eee6989476908feaf64a9dea03e05 Code change is N/A as Nvim has a draw_col variable instead. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | | vim-patch:9.0.1506: line number not displayed when using 'smoothscroll' (#23453)zeertzjq2023-05-03
| | | | | | | | | | | | | | | | | | Problem: Line number not displayed when using 'smoothscroll'. Solution: Adjust condition for showing the line number. (closes vim/vim#12333) https://github.com/vim/vim/commit/88bb3e0a48f160134bdea98cd2b8bd3af86f9d6f
* | | build: create a text file for specifying dependency informationdundargoc2023-05-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | The cmake.deps build will read this file and set the left part of the text as the variable name and the right part as the variable value. The benefit of doing this is that it becomes much easier to parse which dependencies are required, as well as to bump dependencies with scripts/bump_deps.lua. Adjust bump_deps.lua script to work with this new format.
* | | fix(pum): don't position too far with resized parent grid (#23442)zeertzjq2023-05-03
| | |
* | | perf(treesitter): insert/remove items efficiently (#23443)Lewis Russell2023-05-02
| | |
* | | fix(pum): fix missing scrollbar with 'rightleft' (#23448)zeertzjq2023-05-03
| | |
* | | vim-patch:9.0.1505: error when heredoc content looks like heredoc (#23446)zeertzjq2023-05-03
| | | | | | | | | | | | | | | | | | Problem: Error when heredoc content looks like heredoc. Solution: Handle curly expressions. (closes vim/vim#12325) https://github.com/vim/vim/commit/a93d9cdc74f70ca2c85781496ffae4ca738fcd88
* | | fix(pum): fix missing rightmost column with 'rightleft' (#23445)zeertzjq2023-05-02
| | |
* | | Merge pull request #23320 from luukvbaal/smoothscrollzeertzjq2023-05-02
|\ \ \ | | | | | | | | feat(ui): add 'smoothscroll' option
| * | | fix(ui): adjust 'smoothscroll' for inner dimensionsLuuk van Baal2023-05-02
| | | |
| * | | test: 'smoothscroll' <<< marker shows with tabline, winbar and splitsLuuk van Baal2023-05-02
| | | |
| * | | vim-patch:9.0.1502: no test for deleting the end of a long wrapped lineLuuk van Baal2023-05-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for deleting the end of a long wrapped line. Solution: Add a test to check the right text is displayed. (Luuk van Baal, closes vim/vim#12318) https://github.com/vim/vim/commit/5b10a140983c16140d69a214494c4b8af8b34763 Co-authored-by: Luuk van Baal <luukvbaal@gmail.com>