aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * fix(luado): get old_line length before executing Lua codezeertzjq2023-03-04
| |
| * vim-patch:9.0.0016: comparing line pointer for 'breakindent' is not reliablezeertzjq2023-03-04
| | | | | | | | | | | | | | | | | | | | | | Problem: Comparing line pointer for 'breakindent' is not reliable. Solution: Make a copy of the line. https://github.com/vim/vim/commit/c2a79b87fc31080ba24394c0b30bab45f1bea852 Test changes have been squashed into the previous commit. Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * vim-patch:partial:9.0.0013: reproducing memory access errors can be difficultzeertzjq2023-03-04
|/ | | | | | | | | | | | | | | | | | Problem: Reproducing memory access errors can be difficult. Solution: When testing, copy each line to allocated memory, so that valgrind can detect accessing memory before and/or after it. Fix uncovered problems. https://github.com/vim/vim/commit/fa4873ccfc10e0f278dc46f39d00136fab059b19 Since test_override() is N/A, enable ml_get_alloc_lines when ASAN is enabled instead, so it also applies to functional tests. Use xstrdup() to copy the line as ml_line_len looks hard to port. Squash the test changes from patch 9.0.0016. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* vim-patch:9.0.1376: accessing invalid memory with put in Visual block mode ↵zeertzjq2023-03-04
| | | | | | | | | | (#22505) Problem: Accessing invalid memory with put in Visual block mode. Solution: Adjust the cursor column if needed. https://github.com/vim/vim/commit/1c73b65229c25e3c1fd8824ba958f7cc4d604f9c Co-authored-by: Bram Moolenaar <Bram@vim.org>
* build(deps): bump Luv to HEAD - e5da6417d (#22502)Christian Clason2023-03-04
|
* build: unset variables ending with "URL" if USE_EXISTING_SRC_DIR is ONdundargoc2023-03-04
| | | | | | | | | | This will reduce required boilerplate, but more importantly it will automatically unset variables ending on URL. This will help people needing to avoid to unset the URL variable each time a new dependency is added. It is possible that this may remove a non-download variable ending on "URL" in the future, however, the risk of this is likely much lower than the risk of someone forgetting to unset the variable.
* fix(treesitter): raise ts_match_limit to 256 (#22497)Christian Clason2023-03-03
| | | | | | | | | | Problem: Some complex queries may not return all matches. Solution: Raise `ts_match_limit` from current 64 (twice the original default) to 256 (which Helix uses, and seems to be enough for the reported problematic cases). If this leads performance regressions in other queries, we should add a generic querying timeout instead of relying on a low value here.
* fix(treesitter): break early from loop when match is found (#22499)Gregory Anders2023-03-03
| | | Fixup to #22484.
* vim-patch:9.0.0527: long sign text may overflow buffer (#22496)zeertzjq2023-03-03
| | | | | | | | | | | | | | Problem: Long sign text may overflow buffer. Solution: Use a larger buffer. Prevent for overflow. https://github.com/vim/vim/commit/2b1ddf19f8f14365d0b998b4ac12ca85c0923475 Don't change the size of extra[] as it's already large enough. N/A patches for version.c: vim-patch:9.0.0523: more compiler warnings for arguments in small version Co-authored-by: Bram Moolenaar <Bram@vim.org>
* Merge #22493 lua2dox.lua, vim.treesitter tagsJustin M. Keyes2023-03-03
|\
| * docs: lua2dox.lua debuggingJustin M. Keyes2023-03-03
| |
| * docs: fix vim.treesitter tagsJustin M. Keyes2023-03-03
|/ | | | | | | | | Problem: Help tags like vim.treesitter.language.add() are confusing because `vim.treesitter.language` is (thankfully) not a user-facing module. Solution: Ignore the "fstem" when generating "treesitter" tags.
* Merge pull request #22442 from bfredl/quickmakebfredl2023-03-03
|\ | | | | refactor(build): spring cleaning of compile time checks
| * refactor(build): graduate -Wvla, -fno-common and -Og "features"bfredl2023-03-03
| | | | | | | | | | | | These are in gcc 4.9+ which we require anyway. -Og is safe on clang as well (reserved for a future debug-optimize mode, and currently does the same as -O1 which is reasonable)
| * refactor(build): graduate unibilium VAR_FROM feature from 2017bfredl2023-03-03
| |
| * refactor(build): graduate msgpack-c FLOAT32 "feature" since foreverbfredl2023-03-03
| |
| * refactor(build): graduate libtreesitter features which are 1+ years oldbfredl2023-03-03
| |
| * refactor(build): graduate HAVE_LOCALE_H featurebfredl2023-03-03
| | | | | | | | | | | | Merge locale.h into os/lang.h Having a source file with the same name as a system header we use is considered an anti-pattern.
| * refactor(build): remove unused stdlib function and include checksbfredl2023-03-03
| | | | | | | | | | | | In addition: merge some checks for the same feature into one test_compile. This reduces the total number of test compiles which speeds up the cmake configure stage.
* | feat(treesitter): bundle query parser and queries (#22483)Christian Clason2023-03-03
| | | | | | skip injections for now
* | Merge pull request #22484 from gpanders/inspect-tree-fix-cursorChristian Clason2023-03-03
|\ \ | |/ |/| fix(treesitter): maintain cursor position when toggling anonymous nodes
| * refactor(treesitter): use string.format to create linesGregory Anders2023-03-02
| |
| * fix(treesitter): maintain cursor position when toggling anonymous nodesGregory Anders2023-03-02
| | | | | | | | | | | | When toggling anonymous nodes in the :InspectTree window, keep the cursor fixed relative to the node within the tree. This prevents the cursor from jumping.
* | Merge pull request #22494 from zeertzjq/vim-9.0.0531zeertzjq2023-03-03
|\ \ | | | | | | vim-patch:9.0.{partial:0531,0533}: the win_line() function is much too long
| * | vim-patch:9.0.0533: the win_line() function is much too longzeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The win_line() function is much too long. Solution: Move code to separate functions. https://github.com/vim/vim/commit/e49f9acecc03755db850410b2590ad7312c5224b Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | vim-patch:partial:9.0.0531: the win_line() function is much too longzeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The win_line() function is much too long. Solution: Move code to separate functions. https://github.com/vim/vim/commit/c20a41972c61ab4bc211327d535c73c9b30cc166 Partial port as this depends on patch 9.0.0148. Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | refactor(drawline.c): move number column helpers function togetherzeertzjq2023-03-03
|/ /
* | fix(treesitter): typos in _range.luaJaehwang Jung2023-03-03
| | | | | | fix(treesitter): typos _range.lua
* | fix(treesitter): disallow empty filetypesLewis Russell2023-03-03
| | | | | | Fixes #22473
* | vim-patch:9.0.0519: the win_line() function is much too long (#22491)zeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | Problem: The win_line() function is much too long. Solution: Move the code to draw the line number to a separate function. https://github.com/vim/vim/commit/d7657e95b21f660a202a5118af059aa17ad37120 N/A patches for version.c: vim-patch:9.0.0521: compiler warns for unused argument in small version Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | refactor(win_line): move some variables into a struct (#22490)zeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | This reduces the number of arguments to helpers functions. N/A patches for version.c: vim-patch:9.0.0152: warning for unused argument in small build Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | Merge pull request #22488 from zeertzjq/vim-9.0.1373zeertzjq2023-03-03
|\ \ | | | | | | vim-patch:9.0.{0690,0691,0991,1373}: listchars fixes
| * | vim-patch:9.0.1373: wrong text displayed when using both 'linebreak' and 'list'zeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Wrong text displayed when using both 'linebreak' and 'list'. Solution: Only set "c_extra" to NUL when "p_extra" is not empty. (Hirohito Higashi, closes vim/vim#12065) https://github.com/vim/vim/commit/194555c001f2b8576483ef34511450b6e9b5e3fd Cherry-pick a change from patch 9.0.0153. Co-authored-by: h-east <h.east.727@gmail.com>
| * | vim-patch:9.0.0991: crash when reading help index with various options setzeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when reading help index with various options set. (Marius Gedminas) Solution: Do not set wlv.c_extra to NUL when wlv.p_extra is NULL. (closes vim/vim#11651) https://github.com/vim/vim/commit/c67c89c7589253215d57bad588edcf83a9403560 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | vim-patch:9.0.0691: lalloc(0) error in listchars testzeertzjq2023-03-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: lalloc(0) error in listchars test. Solution: Skip generating text for tab if tab_len is zero. https://github.com/vim/vim/commit/2b7b4f7670f607704307f7715ce56752757c22e3 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | vim-patch:9.0.0690: buffer size for expanding tab not correctly computedzeertzjq2023-03-03
|/ / | | | | | | | | | | | | | | | | Problem: Buffer size for expanding tab not correctly computed. Solution: Correctly use size of end character. https://github.com/vim/vim/commit/a0789478f6ebbb823670b7e14ce13ea3fd3b0217 Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | vim-patch:9.0.1371: ballooneval interferes with Insert completion (#22487)zeertzjq2023-03-03
| | | | | | | | | | | | | | Problem: Ballooneval interferes with Insert completion. Solution: Ignore mouse-move events when completing. (closes vim/vim#12094, closes vim/vim#12092) https://github.com/vim/vim/commit/440d4cb55b84fd4b188630abc4a1312598649af0
* | vim-patch:9.0.1368: Bass files are not recognized (#22485)Amaan Qureshi2023-03-02
| | | | | | | | | | | | Problem: Bass files are not recognized. Solution: Add patterns for Bass files. (Amaan Qureshi, closes vim/vim#12088) https://github.com/vim/vim/commit/4ed914b18a47192f79f342bea5e8f59e120d5260
* | build: cmake cleanup (#22251)dundargoc2023-03-02
|/ | | | | | | | - Remove unused code - Use consistent casing. Variable names such as LibLuV_LIBRARIES is needlessly jarring, even if the name might be technically correct. - Use title casing for packages. find_package(unibilium) requires the find_module to be named "Findunibilium.cmake", which makes it harder to spot when scanning the files. Instead, use "Unibilium".
* feat(treesitter): add :InspectTree command (#22477)Christian Clason2023-03-02
|
* refactor: rename show_tree => inspect_tree #22474Justin M. Keyes2023-03-02
| | | | | | | | | | | | | | | | Problem: "show" is potentially a new verb that we can avoid (there is already "open" and "echo"). Even if we can't avoid it, the behavior of `show_tree` fits well in the "inspect" family of functions: a way for users to introspect/reflect on the state of Nvim. Existing "inspect" functions: vim.inspect() vim.inspect_pos() vim.treesitter.inspect_language() nvim__inspect_cell Solution: Rename `show_tree` to `inspect_tree`.
* build: remove libfindmacros library (#22423)dundargoc2023-03-02
| | | | | | Large parts the library weren't being used, and the parts that were was overly abstracted for our use case. Additionally, part of its use case was to abstract pkgconfig boilerplate, which is no longer needed as pkgconfig has been removed in favor of relying on cmake alone in 09118052cee5aef978d6075db5287c1b6c27381a.
* test(treesitter/parser_spec): correct time unit (#22471)zeertzjq2023-03-02
|
* feat(vim.fs): pass path to find() predicate, lazy evaluate #22378Mike2023-03-01
| | | | | | | | Problem: No easy way to find files under certain directories (ex: grab all files under `test/`) or exclude the content of certain paths (ex. `build/`, `.git/`) Solution: Pass the full `path` as an arg to the predicate.
* fix(lsp): only fire LspDetach for attached buffers (#22468)Gregory Anders2023-03-01
| | | | | | | | | | If the LSP server fails to start then the client never initializes and thus never calls its on_attach function and an LspAttach event is never fired. However, the on_exit function still fires a LspDetach event, so user autocommands that attempt to "clean up" in LspDetach may run into problems if they assume that the buffer was already attached. The solution is to only fire an LspDetach event if the buffer was already attached in the first place.
* vim-patch:8.2.{1949,2781} (#22451)zeertzjq2023-03-01
| | | | | | | | | | | | | | | | | | | | | | | | | | vim-patch:8.2.2781: add() silently skips when adding to null list or blob Problem: Add() silently skips when adding to null list or blob. Solution: Give an error in Vim9 script. Allocate blob when it is NULL like with list and dict. https://github.com/vim/vim/commit/b7c21afef14bba0208f2c40d47c050a004eb2f34 Do not implicitly change a NULL blob/dict/list to an empty one. N/A patches for version.c: vim-patch:8.2.1949: Vim9: using extend() on null dict is silently ignored Problem: Vim9: using extend() on null dict is silently ignored. Solution: Give an error message. Initialize a dict variable with an empty dictionary. (closes vim/vim#7251) https://github.com/vim/vim/commit/348be7ed07d164970ec0004bc278e254eb0cf5bf N/A because Nvim's current behavior is an error message as a locked list/dict, which is more consistent. Ref #4615. Co-authored-by: Bram Moolenaar <Bram@vim.org>
* fix(lsp): use buffer scheme for files not stored on disk (#22407)Mathias Fußenegger2023-03-01
| | | | | | Sending `didOpen` with a `file` scheme causes problems with some language servers because they expect the file to exist on disk. See https://github.com/microsoft/language-server-protocol/pull/1679
* test(ui): wait for another success with failure after successzeertzjq2023-03-01
| | | | | | | | | | | Problem: In a success-failure-success situation, if minimal timeout is reached between the failure and the second success, the session is stopped without waiting for the second success, causing the test to fail. Solution: Wait for another success if a failure is seen after a success. Ref #22155 #22464
* fix(lsp): callHierarchy methods also require the callHierarchyProvider (#22427)Jens Claes2023-03-01
|
* test(termxx_spec): fix TermClose bdelete test flakiness (#22463)zeertzjq2023-03-01
| | | | | | | | Problem: If shell-test finishes before the next RPC call, TermClose has already been triggered, so the test fails. Solution: Add INTERACT argument so that shell-test keeps running.