aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* build: bump lpeg to 1.1.0 (#25016)dundargoc2023-09-04
| | | | Release notes indicates it has better UTF8 handling which is relevant for us.
* vim-patch:9.0.1863: wrong format specifiers in e_aptypes_is_null_str_nr (#25015)zeertzjq2023-09-04
| | | | | | | | Problem: wrong format specifiers in e_aptypes_is_null_str_nr Solution: Fix the wrong format specifier closes: vim/vim#13020 https://github.com/vim/vim/commit/7db89bdc23e53c7bc43af6f1c7281bc69a6a3098
* vim-patch:9.0.1856: issues with formatting positional arguments (#25013)zeertzjq2023-09-04
| | | | | | | | | | | | | Problem: issues with formatting positional arguments Solution: fix them, add tests and documentation closes: vim/vim#12140 closes: vim/vim#12985 Tentatively fix message_test. Check NULL ptr. https://github.com/vim/vim/commit/aa90d4f031f73a34aaef5746931ea746849a2231 Co-authored-by: Christ van Willegen <cvwillegen@gmail.com>
* build: various fixesdundargoc2023-09-04
| | | | | | | | | - simplify lua interpreter search - fix incorrect variable name in BuildLua.cmake - build PUC Lua with -O2 - silence non-mandatory find_package search for libuv - simplify Find modules - Prefer using the explicitly set CI_BUILD over relying on the environment variable "CI".
* Merge pull request #24974 from bfredl/memline2bfredl2023-09-03
|\ | | | | fix(memline): more curbuf stuff
| * fix(memline): more curbuf stuffbfredl2023-09-01
| |
* | ci: move external_deps job from cirrus-ci back to github actionsdundargoc2023-09-03
| | | | | | | | | | | | | | This partially reverts commit 7d0479c55810af9bf9f115ba69d1419ea81ec41e. The job has been particularly unstable when used with docker on cirrus-ci, which is especially bad as it's meant to be a non-flaky and simple test.
* | refactor(ex_session.c): remove unnecessary char -> int -> char castzeertzjq2023-09-03
| | | | | | | | The two calls to get_view_file() both pass a char in a string, and get_view_file() assigns it to a char in a string.
* | vim-patch:9.0.1848: [security] buffer-overflow in vim_regsub_both() (#25001)zeertzjq2023-09-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: buffer-overflow in vim_regsub_both() Solution: Check remaining space https://github.com/vim/vim/commit/ced2c7394aafdc90fb7845e09b3a3fee23d48cb1 The change to do_sub() looks confusing. Maybe it's an overflow check? Then the crash may not be applicable to Nvim because of different casts. The test also looks confusing. It seems to source itself recursively. Also don't call strlen() twice on evaluation result. N/A patches for version.c: vim-patch:9.0.1849: CI error on different signedness in ex_cmds.c vim-patch:9.0.1853: CI error on different signedness in regexp.c Co-authored-by: Christian Brabandt <cb@256bit.org>
* | perf(substitute): don't reallocate new_start every time (#24997)zeertzjq2023-09-03
| |
* | vim-patch:9.0.1840: [security] use-after-free in do_ecmd (#24993)zeertzjq2023-09-03
| | | | | | | | | | | | | | | | | | | | | | Problem: use-after-free in do_ecmd Solution: Verify oldwin pointer after reset_VIsual() https://github.com/vim/vim/commit/e1dc9a627536304bc4f738c21e909ad9fcf3974c N/A patches for version.c: vim-patch:9.0.1841: style: trailing whitespace in ex_cmds.c Co-authored-by: Christian Brabandt <cb@256bit.org>
* | refactor(marks): don't set timestamp twice with :delmarks (#24994)zeertzjq2023-09-03
| | | | | | refactor(mark): don't set same timestamp twice
* | fix(shada): update marks when using delmarks! (#24978)Maria José Solano2023-09-03
| |
* | Merge pull request #24991 from zeertzjq/vim-9.0.1846zeertzjq2023-09-03
|\ \ | | | | | | vim-patch:9.0.{1846,1847}
| * | vim-patch:9.0.1847: [security] potential oob write in do_addsub()zeertzjq2023-09-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: potential oob write in do_addsub() Solution: don't overflow buf2, check size in for loop() https://github.com/vim/vim/commit/889f6af37164775192e33b233a90e86fd3df0f57 Co-authored-by: Christian Brabandt <cb@256bit.org>
| * | vim-patch:9.0.1846: [security] crash in fullcommandzeertzjq2023-09-03
|/ / | | | | | | | | | | | | | | | | Problem: crash in fullcommand Solution: Check for typeval correctly https://github.com/vim/vim/commit/4c6fe2e2ea62469642ed1d80b16d39e616b25cf5 Co-authored-by: Christian Brabandt <cb@256bit.org>
* | vim-patch:9.0.1852: i_CTRL-O does not reset Select Mode (#24990)zeertzjq2023-09-03
| | | | | | | | | | | | | | | | | | | | | | | | Problem: i_CTRL-O does not reset Select Mode Solution: Reset select mode on CTRL-O in insert mode closes: vim/vim#13001 closes: vim/vim#12115 https://github.com/vim/vim/commit/d69aecf141ff05a645d02f39f1cbf6381ed7d0c0 Co-authored-by: pierreganty <pierreganty@gmail.com> Co-authored-by: Christian Brabandt <cb@256bit.org>
* | build: download busted from own neovim/deps repositorydundargoc2023-09-03
| | | | | | | | | | | | | | | | Downloading the necessary files all at once instead of doing dependency handling with luarocks speeds up installation immensely. We speed up the process even more by using luv as a replacement for the C modules in the busted dependencies, which allows us to skip costly compilation times. Co-authored-by: bfredl <bjorn.linse@gmail.com>
* | docs(luv): correct uv.spawn options.args docs about the first argumentChristian Clason2023-09-02
| | | | | | | | https://github.com/luvit/luv/commit/045bf29b6f54f9b511f8b76088250d2a90dbdf4f
* | fix(ui): avoid ambiguity about chunk that clears part of line (#24982)zeertzjq2023-09-02
| | | | | | Co-authored-by: bfredl <bjorn.linse@gmail.com>
* | Merge pull request #24310 from lewis6991/refactor/optionvalidateLewis Russell2023-09-02
|\ \ | | | | | | refactor(option.c): misc
| * | refactor(option): add set_option()Lewis Russell2023-08-31
| | |
| * | refactor(option): pass varp to set_string_optionLewis Russell2023-08-31
| | |
| * | refactor(option): option clearingLewis Russell2023-08-31
| | |
| * | refactor(option): remove OPT_CLEARLewis Russell2023-08-31
| | |
| * | refactor(option): further align set_string_option with do_set_option_string (2)Lewis Russell2023-08-31
| | |
| * | refactor(option): further align set_string_option with do_set_option_stringLewis Russell2023-08-31
| | |
| * | refactor(option): remove hidden option checkLewis Russell2023-08-31
| | |
| * | refactor(option): remove redundant localLewis Russell2023-08-31
| | |
| * | refactor(option): change some int to boolLewis Russell2023-08-31
| | |
| * | refactor(option.c): call did_set_option for all typesLewis Russell2023-08-31
| | | | | | | | | | | | | | | | | | | | | | | | set_option_value() only called did_set_option() for string options, whereas do_set_option_value() called it for all types. This change makes set_option_value() call did_set_option() for all types and thus makes it more consistent with do_set_option_value().
| * | fix(optionstr.c): incorrect use of curbuf/curwinLewis Russell2023-08-31
| | |
| * | refactor(optionstr.c): remove redundant argumentLewis Russell2023-08-31
| | |
| * | refactor(option.c): remove did_set_string_option aliasLewis Russell2023-08-31
| | |
| * | refactor(option.c): move validation logic to functionLewis Russell2023-08-31
| | |
* | | vim-patch:acb91d3905cfChristian Clason2023-09-02
| | | | | | | | | | | | | | | | | | | | | | | | runtime(zserio): add zserio syntax (vim/vim#13005) https://github.com/vim/vim/commit/acb91d3905cfef5eff8edfb76e62a6b6bab1e91e Co-authored-by: Dominique Pellé <dominique.pelle@gmail.com>
* | | vim-patch:da16a1b471aaChristian Clason2023-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | runtime(ruby): Update syntax, indent and ftplugin files While making changes to the ruby ftplugin, slightly change the exepath() conditional from patch 9.0.1833 and move it after the :cd invocation. closes: 12981 closes: 12994 https://github.com/vim/vim/commit/da16a1b471aa717f58909cc6531cb6dbbff14d22 Co-authored-by: Doug Kearns <dougkearns@gmail.com> Co-authored-by: Tim Pope <code@tpope.net>
* | | test(shada/marks_spec): load the file with the marks (#24979)zeertzjq2023-09-02
| |/ |/|
* | fix(lsp): wrong iterator in registerCapability handler (#24971)TheBlob422023-09-01
| |
* | vim-patch:9.0.1833: [security] runtime file fixes (#24969)zeertzjq2023-09-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: runtime files may execute code in current dir Solution: only execute, if not run from current directory The perl, zig and ruby filetype plugins and the zip and gzip autoload plugins may try to load malicious executable files from the current working directory. This is especially a problem on windows, where the current directory is implicitly in your $PATH and windows may even run a file with the extension `.bat` because of $PATHEXT. So make sure that we are not trying to execute a file from the current directory. If this would be the case, error out (for the zip and gzip) plugins or silently do not run those commands (for the ftplugins). This assumes, that only the current working directory is bad. For all other directories, it is assumed that those directories were intentionally set to the $PATH by the user. https://github.com/vim/vim/commit/816fbcc262687b81fc46f82f7bbeb1453addfe0c Co-authored-by: Christian Brabandt <cb@256bit.org>
* | vim-patch:9.0.1828: cursor wrong with virt text before double-width char ↵zeertzjq2023-09-01
| | | | | | | | | | | | | | | | | | | | | | (#24967) Problem: Wrong cursor position with virtual text before double-width char at window edge. Solution: Check for double-width char before adding virtual text size. closes: vim/vim#12977 https://github.com/vim/vim/commit/ac2d8815ae7a93c54b07cba76475cfb3f26a3ac6
* | fix: use snprintf instead of sprintfLewis Russell2023-08-31
| | | | | | | | Clang 14 now reports sprintf as deprecated.
* | fix(query_error): multiline bugLewis Russell2023-08-31
|/
* feat(treesitter): improve query error messageAmaan Qureshi2023-08-31
|
* vim-patch:0b8b145bf8bfChristian Clason2023-08-31
| | | | | | | | | | runtime(optwin): Fix for 'splitkeep' option (vim/vim#12974) 'spk' was used as a boolean, rather than a string option. https://github.com/vim/vim/commit/0b8b145bf8bfd3e90a1c30a999e6adb89ec8891c Co-authored-by: xrandomname <141588647+xrandomname@users.noreply.github.com>
* vim-patch:1610528cc305Christian Clason2023-08-31
| | | | | | | | runtime(forth): Update syntax and ftplugin files (vim/vim#12976) https://github.com/vim/vim/commit/1610528cc3052103e368c4175b09db6f9a6c150c Co-authored-by: dkearns <dougkearns@gmail.com>
* refactor(lsp): add type annotation for lsp.Client.server_capabilities (#24925)Jongwook Choi2023-08-31
| | | | | | | | | | | | | | The class `lsp.Client` has a public member `server_capabilities`, which is assumed to be non-nil once initialized, as documented in `:help vim.lsp.client`. Due to the possibility that it may be nil before initialization, `lsp.Client` was not having a proper lua type annotations on the field `server_capabilities`. Instead of having a nil `server_capabilities` until initialized in the RPC response callback, we can have an initial value of empty table. This CHANGES the behavior of the `server_capabilities` field in a way that it is no longer `nil` until initialization. Note that, as already documented, `server_capabilities` should never be nil when it is once initialized and thus ready to be used in user configs.
* fix(lsp): only disable inlay hints / diagnostics if no other clients are ↵Chris AtLee2023-08-31
| | | | | | | | | | | | | | | | | | | connected (#24535) This fixes the issue where the LspNotify handlers for inlay_hint / diagnostics would end up refreshing all attached clients. The handler would call util._refresh, which called vim.lsp.buf_request, which calls the method on all attached clients. Now util._refresh takes an optional client_id parameter, which is used to specify a specific client to update. This commit also fixes util._refresh's handling of the `only_visible` flag. Previously if `only_visible` was false, two requests would be made to the server: one for the visible region, and one for the entire file. Co-authored-by: Stanislav Asunkin <1353637+stasjok@users.noreply.github.com> Co-authored-by: Mathias Fußenegger <mfussenegger@users.noreply.github.com>
* fix(shada): update deleted marks (#24936)Maria José Solano2023-08-31
| | | | | | Fix #4295 Close #16067 Co-authored-by: chentau <tchen1998@gmail.com>
* fix(runtime/tutor): don't try to close fold when there is none (#24953)David Moberg2023-08-31
| | | | | | Problem: When double clicking a line starting with a #, the code assumes there is a fold there and tries to close it, resulting in an error if there isn't a fold. Solution: Check foldlevel before performing "zc".