aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | vim-patch:partial 2d8ed0203aed (#18675)dundargoc2022-05-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * vim-patch:partial 2d8ed0203aed Update runtime files. https://github.com/vim/vim/commit/2d8ed0203aedd5f6c22efa99394a3677c17c7a7a Skip: runtime/doc/map.txt runtime/doc/syntax.txt runtime/doc/usr_51.txt runtime/doc/usr_52.txt runtime/syntax/help.vim runtime/syntax/vim.vim Co-authored-by: Sean Dewar <seandewar@users.noreply.github.com>
* | | vim-patch:8.2.5009: fold may not be closeable after appending (#18722)Brandon Simmons2022-05-23
| | | | | | | | | | | | | | | Problem: Fold may not be closeable after appending. Solution: Set the fd_small flag to MAYBE. (Brandon Simmons, closes vim/vim#10471) https://github.com/vim/vim/commit/e8c4a64bffbe628a46dc172d04cfc2db6e8dd8b6
* | | Merge pull request #18712 from famiu/feat/ui/winbar_redrawbfredl2022-05-23
|\ \ \ | | | | | | | | fix(ui): redraw winbar alongside statusline
| * | | fix(ui): redraw winbar alongside statuslineFamiu Haque2022-05-23
| | | | | | | | | | | | | | | | | | | | Remove `w_redr_winbar` and use `w_redr_status` to redraw the winbar to ensure that winbar redraw is triggered alongside the statusline redraw.
* | | | vim-patch:8.2.5008: when 'formatoptions' contains "/" wrongly wrapping ↵zeertzjq2022-05-23
|/ / / | | | | | | | | | | | | | | | | | | | | | comment (#18717) Problem: When 'formatoptions' contains "/" wrongly wrapping a long trailing comment. Solution: Pass the OPENLINE_FORMAT flag. https://github.com/vim/vim/commit/7e667788150be617aeac42b0d668618ac33ab9da
* | | feat: click support for 'statusline', 'winbar' #18650Famiu Haque2022-05-23
| | | | | | | | | | | | The mouseclick item "%@" is now supported by 'statusline' and 'winbar'. Previously it was only supported by 'tabline'.
* | | refactor!: delete insertmode (#18547)Gregory Anders2022-05-22
| | | | | | | | | | | | | | | | | | | | | Neovim already removed `evim` (or any similar flags). The 'insertmode' option is a weird remnant, so get rid of it. The 'insertmode' option is replaced with a script that closely emulates the option. This script is documented at :help 'insertmode'
* | | Merge pull request #18691 from zeertzjq/vim-8.2.4996zeertzjq2022-05-23
|\ \ \ | | | | | | | | vim-patch:8.2.{4996.5002}: setbufline(), deletebufline() may change Visual selection
| * | | vim-patch:8.2.5002: deletebufline() may change Visual selectionzeertzjq2022-05-23
| | | | | | | | | | | | | | | | | | | | | | | | Problem: deletebufline() may change Visual selection. Solution: Disable Visual mode when using another buffer. (closes vim/vim#10469) https://github.com/vim/vim/commit/9b2edfd3bf2f14a1faaee9b62930598a2e77a798
| * | | vim-patch:8.2.4996: setbufline() may change Visual selectionzeertzjq2022-05-23
| | | | | | | | | | | | | | | | | | | | | | | | Problem: setbufline() may change Visual selection. (Qiming Zhao) Solution: Disable Visual mode when using another buffer. (closes vim/vim#10466) https://github.com/vim/vim/commit/0ad00a7fd3e0389f565876521e395c35144d8009
* | | | vim-patch:8.2.5001: checking translations affects the search pattern history ↵zeertzjq2022-05-23
|/ / / | | | | | | | | | | | | | | | | | | (#18709) Problem: Checking translations affects the search pattern history. Solution: Use "keeppatterns". (Doug Kearns) https://github.com/vim/vim/commit/8a3704723c40779d688ef957dbe5bd8b65c25f95
* | | fix(lsp): do not detach LSP servers on Windows #18703Gregory Anders2022-05-22
| | | | | | | | | | | | | | | | | | | | | Detaching the process seems to have unintended side effects on Windows, so only do it by default on non-Windows platforms. Ref: https://github.com/neovim/nvim-lspconfig/issues/1907 Closes https://github.com/neovim/nvim-lspconfig/pull/1913
* | | refactor(runtime): convert the remaining dist#ft functions to lua (#18623)Jonas Strittmatter2022-05-22
| | |
* | | build(clint): remove redundant checks #18698dundargoc2022-05-22
| | | | | | | | | | | | | | | Uncrustify is the source of truth where possible. Remove any redundant checks from clint.py. See also https://github.com/neovim/neovim/pull/18563
* | | Merge pull request #18646 from bfredl/winbarrulerbfredl2022-05-22
|\ \ \ | | | | | | | | fix(winbar): allow winbar to display a ruler
| * | | fix(winbar): allow winbar to display the rulerbfredl2022-05-22
| | | |
* | | | fix(cid/352839): USE_AFTER_FREE in create_user_command #18669James McCoy2022-05-22
| | | |
* | | | Merge pull request #18608 from ckipp01/extmarkDocsbfredl2022-05-22
|\ \ \ \ | |/ / / |/| | | docs(extmark): remove message about passing in id=0
| * | | docs(extmark): remove message about passing in id=0ckipp012022-05-18
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The docs for `nvim_buf_set_extmark` mention that you can create a new extmark when passing in `id=0`, however if you do this you'll get an error since the code checks that id is positive. ``` id is not a positive integer ``` This change re-words the sentence to make it clearer.
* | | fix(mksession): don't store floats in session #18635Daniel Steinberg2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: If there are floating windows when `:mksession` runs, the session cannot be properly restored. Solution: Change `:mksession` to skip floating windows. This matches Vim's treatment of popup windows. An alternative approach could have `:mksession` save floating windows that can be _properly_ restored (rather than skip them entirely, which is what this PR does). While that would seemingly be a more complete fix, that could present additional issues since floating windows are ordinarily created by plugins, and they may no longer be properly under a plugin's control when restored. closes #18432
* | | Merge pull request #18696 from bfredl/quickincbfredl2022-05-22
|\ \ \ | | | | | | | | fix(tests): fix some issues with ui/inccommand_spec.lua causing slowness
| * | | fix(tests): fix some issues with ui/inccommand_spec.lua causing slownessbfredl2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | from ~30 to ~20 secs with ASAN build - feedkeys test: the same substitute was repeated, and not even correctly spelled - don't clear() in a tight loop. "bwipe!" is enough to erase undo history - error in cnoremap mapping causes a check_for_delay(). <c-c> it away.
* | | | ci(commitlint): only trigger on PRs to master #18679dundargoc2022-05-22
| | | | | | | | | | | | | | | | This will prevent a failure for backported PRs as they use a different commit message.
* | | | Merge pull request #18693 from bfredl/quicklspbfredl2022-05-22
|\ \ \ \ | |/ / / |/| | | fix(ci): remove 2000ms blocking wait in many plugin/lsp_spec.lua tests
| * | | fix(ci): remove 2000ms blocking wait in many plugin/lsp_spec.lua testsbfredl2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cuts down typical run time for `plugin/lsp_spec.lua` from 70 secs to 12 secs in ASAN CI build. This happens in ASAN/EXIT_FREE builds where nvim waits 2000ms due to unclosed handled. I wasn't able to pin-point the exact cause. But these tests ran in nested context where two server/client pairs were setup for no good reason. Moving these tests out so only one client is being setup fixed the exit hang.
* | | | vim-patch:8.2.4935: with 'foldmethod' "indent" some lines not included in ↵zeertzjq2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fold (#18694) Problem: With 'foldmethod' "indent" some lines are not included in the fold. (Oleg Koshovetc) Solution: Fix it. (Brandon Simmons, closes vim/vim#10399, closes vim/vim#3214) https://github.com/vim/vim/commit/d98e75e23666c159c7e00bcf5b6ad9a933bb0534
* | | | fix(folds): fix fold regression with :move (#18685)zeertzjq2022-05-22
| | | |
* | | | Merge pull request #18681 from bfredl/uitestbfredl2022-05-22
|\ \ \ \ | | | | | | | | | | fix(tests): fix some screen.lua warnings
| * | | | fix(tests): fix some screen.lua warningsbfredl2022-05-21
| | | | |
* | | | | docs: remove "Vi default" #18672dundargoc2022-05-22
| |/ / / |/| | | | | | | It's not relevant and clutters the documentation.
* | | | vim-patch:8.2.4993: smart/C/lisp indenting is optional (#18684)zeertzjq2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | Problem: smart/C/lisp indenting is optional, which makes the code more complex, while it only reduces the executable size a bit. Solution: Graduate FEAT_CINDENT, FEAT_SMARTINDENT and FEAT_LISP. https://github.com/vim/vim/commit/8e145b82464a21ee4fdf7948f04e2a1d505f8bfa
* | | | vim-patch:8.2.4987: after deletion a small fold may be closable (#18683)Brandon Simmons2022-05-22
| | | | | | | | | | | | | | | | | | | | Problem: After deletion a small fold may be closable. Solution: Check for a reverse range. (Brandon Simmons, closes vim/vim#10457) https://github.com/vim/vim/commit/3fcccf94e8bc142d2c79c3b62087145896df6b36
* | | | Merge pull request #18682 from zeertzjq/vim-8.1.0535zeertzjq2022-05-22
|\ \ \ \ | | | | | | | | | | vim-patch:8.1.0535,8.2.4991: increment/decrement might get interrupted by updating folds
| * | | | vim-patch:8.2.4991: no test for hwat patch 8.1.0535 fixeszeertzjq2022-05-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for hwat patch 8.1.0535 fixes. Solution: Add a test. (closes vim/vim#10462) https://github.com/vim/vim/commit/93c1573dd284dc9cf5ed9265a0143aaf520d2920
| * | | | vim-patch:8.1.0535: increment/decrement might get interrupted by updating foldszeertzjq2022-05-22
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Increment/decrement might get interrupted by updating folds. Solution: Disable fold updating for a moment. (Christian Brabandt, closes vim/vim#3599) https://github.com/vim/vim/commit/6b731886ca94d66b9bdedfb7e603af44a6400399
* | | | Merge pull request #13729 from devbhansingh/ui_enter_promptbfredl2022-05-21
|\ \ \ \ | |/ / / |/| | | feat(ui): clear message history explicitly with msg_history_clear event
| * | | feat(ui): clear message history explicitly with msg_history_clear eventdevbhan singh2022-05-21
|/ / /
* | | ci: fix lint errors caused by missing includes (#18673)dundargoc2022-05-21
| | | | | | | | | | | | This will fix the `check-single-includes` check which started failing after https://github.com/neovim/neovim/pull/18663/.
* | | fix(autocmds): separate command from desc (#18617)kylo2522022-05-21
| | |
* | | refactor: add warnings for deprecated functions (#18662)dundargoc2022-05-21
| | |
* | | ci: bump github actions versions #18616dundargoc2022-05-20
| | |
* | | build(clint.py): remove rules for includes, whitespace, tabs #18611dundargoc2022-05-20
| | | | | | | | | | | | | | | | | | | | | Uncrustify is the source of truth where possible. Remove any redundant checks from clint.py. See also https://github.com/neovim/neovim/pull/18563 Closes https://github.com/neovim/neovim/issues/17763
* | | ci: lint with uncrustify #18563dundargoc2022-05-20
| | | | | | | | | | | | | | | | | | | | | This lint job will ensure that the C codebase is properly formatted at all times. This helps eliminate most of clint.py. To save CI time, it's faster to manually compile uncrustify and cache the binary instead of using homebrew (the apt-get package is too old).
* | | refactor: remove unused USE_WCHAR_FUNCTIONS #18618Lewis Russell2022-05-20
| | | | | | | | | | | | USE_WCHAR_FUNCTIONS is never defined and we don't trust libc wchar functions anyway.
* | | fix(unittests): coredump when running unit tests #18663Jun-ichi TAKIMOTO2022-05-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fs_init() must be called before early_init() in init/helpers.lua If I run 'make unittest' on my Mac (macOS 10.14/Mojave or 12/Big Sur, intel CPU), every test produce a core dump. Call sequence in the core is: early_init() main.c:197 set_init_1() option.c:508 runtimepath_default() runtime.c:1205 get_lib_dir() runtime.c:1175 os_isdir() fs.c:137 os_getperm() fs.c:777 os_stat() fs.c:761 fs_loop_lock() fs.c:72 uv_mutex_lock(&fs_loop_mutex) thread.c:352 abort() .deps/build/src/libuv/src/unix/thread.c: void uv_mutex_lock(uv_mutex_t* mutex) { if (pthread_mutex_lock(mutex)) abort(); // line 352 } So pthread_mutex_lock(&fs_loop_mutex) failed. The reason seems to be simple. fs_init() was not called and fs_loop_mutex has not been initialized. fs_init() was moved out from early_init() in main.c by b87867e69e94d9784468a126f21c721446f080de, but unit/helpers.lua was not updated accordingly.
* | | Merge pull request #18648 from jamessan/coverity-fixesJames McCoy2022-05-20
|\ \ \
| * | | ci(coverity): run every day instead of once/weekJames McCoy2022-05-20
| | | | | | | | | | | | | | | | | | | | Neovim currently falls in the 100 kLOC - 500 kLOC bucket, which can scan 3x/day.
| * | | perf(cid/350479): avoid copying ExtmarkInfo when calling extmark_to_array()James McCoy2022-05-20
| | | |
| * | | fix(cid/352782): assert str->items is non-NULL to hint static analyzersJames McCoy2022-05-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The earlier vsnprintf() call checks whether str->items is NULL, sets of the "possible NULL" spidey sense. kv_ensure_space() guarantees str->items is non-NULL but since it doesn't use NULL checks to decide whether to alloc, static analyzers can't tell this code path is safe.
| * | | fix(cid/348314): free user_copy, not userJames McCoy2022-05-20
| | | | | | | | | | | | | | | | | | | | user is passed in by the caller, which we internally copy. We should be freeing our copy, not the caller's string.