aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | vim-patch:8.2.1260: there is no good test for CursorHold (#21086)zeertzjq2022-11-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: There is no good test for CursorHold. Solution: Add a test. Remove duplicated test. (Yegappan Lakshmanan, closes vim/vim#6503 https://github.com/vim/vim/commit/7591116acffc45b5880c49244646651badac1629
| * | | | | | ci(release/winget): correctly set output variable (#21077)Vedant2022-11-17
| | | | | | |
| * | | | | | ci(windows): set $PSNativeCommandArgumentPassing = 'Legacy' (#21083)zeertzjq2022-11-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ref https://github.com/actions/runner-images/issues/6586 Some runners are using new images, while some others are using old image. This is the only way I can find that makes it work on both.
| * | | | | | vim-patch:9.0.0300: 'cpoptions' tests are flaky (#21081)zeertzjq2022-11-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: 'cpoptions' tests are flaky. Solution: Use a different file name for each test. https://github.com/vim/vim/commit/a85e4db9780a4cf7a72cbb98c7127922f668cdf6 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | fix(api): nvim_win_set_cursor redraw cursorcolumn for non-current window ↵Jlll12022-11-17
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#21072) fix #19063 this fixes the cursorcolumn not being redrawn for non-current windows in `nvim_win_set_cursor()`
| * | | | | ci(stale): disable remove-stale-when-updated (#21075)zeertzjq2022-11-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The needs:response label should only be added and removed manually, and the action's behavior of removing the label on any activity (e.g. title change, removing reviewers) is unwanted.
| * | | | | fix(api): nvim_buf_get_text regression (#21071)Lewis Russell2022-11-15
| | | | | |
| * | | | | refactor: convert drawline.c draw states to enum (#21067)luukvbaal2022-11-15
| | | | | |
| * | | | | vim-patch:9.0.0884: mouse shape remains in op-pending mode after failed ↵zeertzjq2022-11-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | change (#21066) Problem: Mouse shape remains in op-pending mode after failed change. Solution: Reset finish_op and restore it. (closes vim/vim#11545) https://github.com/vim/vim/commit/cdeb65729d96c90320b9009e583ade305c396f29
| * | | | | test: don't skip parser_spec on windows (#20294)dundargoc2022-11-15
| | | | | |
| * | | | | Merge pull request #20157 from dundargoc/build/iwyubfredl2022-11-15
| |\ \ \ \ \ | | | | | | | | | | | | | | build: allow IWYU to fix includes for all .c files
| | * | | | | build: allow IWYU to fix includes for all .c filesdundargoc2022-11-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow Include What You Use to remove unnecessary includes and only include what is necessary. This helps with reducing compilation times and makes it easier to visualise which dependencies are actually required. Work on https://github.com/neovim/neovim/issues/549, but doesn't close it since this only works fully for .c files and not headers.
| * | | | | | Merge pull request #21031 from dundargoc/fix/windowsbfredl2022-11-15
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | ci(windows): treat compiler warnings as errors
| | * | | | | ci(windows): treat compiler warnings as errorsdundargoc2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce the warning level from 3 to 1 and fix all warnings.
| * | | | | | docs(treesitter): change links for `eq?` and `set!` to codeblocks (#21047)Jlll12022-11-15
| | | | | | |
| * | | | | | vim-patch:9.0.0882: using freed memory after SpellFileMissing autocmd uses ↵zeertzjq2022-11-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bwipe (#21060) Problem: Using freed memory after SpellFileMissing autocmd uses bwipe. Solution: Bail out if the window no longer exists. https://github.com/vim/vim/commit/c3d27ada14acd02db357f2d16347acc22cb17e93 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | vim-patch:9.0.0883: a silent mapping may cause dots on the command line (#21061)zeertzjq2022-11-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: A silent mapping may cause dots on the command line. Solution: Don't show dots for completion if they are not going to be removed again. (closes vim/vim#11501) https://github.com/vim/vim/commit/698a00f55d60043d51b1c98cbbf3f9fd10badd2f Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | Merge pull request #21054 from max397574/fix/deepcopy_vimNILbfredl2022-11-14
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | fix(lua): make `vim.deepcopy` work with `vim.NIL`
| | * | | | | fix(lua): make `vim.deepcopy` work with `vim.NIL`Max2022-11-14
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | style: changed double quotes to single quotes feat: add tests fix tests
| * | | | | feat(lua-api): avoid unnecessary allocations (#19877)Lewis Russell2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | Lua makes (or reuses) an internal copy of strings, so we can safely push buf pointers onto the stack.
| * | | | | vim-patch:9.0.0878: Coverity warns for dead code (#21053)zeertzjq2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Coverity warns for dead code. Solution: Remove the dead code. https://github.com/vim/vim/commit/b298fe6cbae3b240b10dbd55d9c38d0cc8e033d3 Nvim has refactored this function and does not have the dead code. Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | feat(test): add Lua forms for API methods (#20152)Lewis Russell2022-11-14
| | | | | |
| * | | | | test: remove skip for 32-bit MSVC (#21030)dundargoc2022-11-14
| | | | | | | | | | | | | | | | | | We don't support 32-bit windows anymore so it's not needed.
| * | | | | vim-patch:8.2.4038: various code not used when features are disabled (#21049)zeertzjq2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Various code not used when features are disabled. Solution: Add #ifdefs. (Dominique Pellé, closes vim/vim#9491) https://github.com/vim/vim/commit/748b308eebe8d8860888eb27da08333f175d547d N/A patches for version.c: vim-patch:8.2.2186: Vim9: error when using 'opfunc' Problem: Vim9: error when using 'opfunc'. Solution: Do not expect a return value from 'opfunc'. (closes vim/vim#7510) https://github.com/vim/vim/commit/5b3d1bb0f5180266c4de4d815b3ea856a2fb3519
| * | | | | vim-patch:8.2.4001: insert complete code uses global variableszeertzjq2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insert complete code uses global variables. Solution: Make variables local to the file and use accessor functions. (Yegappan Lakshmanan, closes vim/vim#9470) https://github.com/vim/vim/commit/d94fbfc74a8b8073e7a256c95fa6f39fc527c726 Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
| * | | | | test(old): add missing lines from Vim patch 8.2.0522 (#21048)zeertzjq2022-11-14
| | | | | |
| * | | | | vim-patch:9.0.0872: code is indented more than needed (#21046)zeertzjq2022-11-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code is indented more than needed. Solution: Return early. (Yegappan Lakshmanan, closes vim/vim#11538) https://github.com/vim/vim/commit/623e94e13810e109c6aa201bcf3a8278429502f3 Only port the first change to init_history() as Nvim has refactored it. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
| * | | | | test: introduce skip() #21010dundargoc2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is essentially a convenience wrapper around the `pending()` function, similar to `skip_fragile()` but more general-purpose. Also remove `pending_win32` function as it can be replaced by `skip(iswin())`.
| * | | | | Merge pull request #21033 from bfredl/winfixbfredl2022-11-13
| |\ \ \ \ \ | | | | | | | | | | | | | | fix(ui): fix some cases of stale highlight definitions
| | * | | | | fix(ui): fix some cases of stale highlight definitionsbfredl2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes #20695
| * | | | | | Merge pull request #20984 from notomo/fix-message-kind-on-historybfredl2022-11-13
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | fix(ui-ext): correct message kind in history before vim.ui_attach()
| | * | | | | | fix(ui-ext): correct message kind in history before vim.ui_attach()notomo2022-11-07
| | | | | | | |
| * | | | | | | vim-patch:9.0.0865: duplicate arguments are not always detected (#21036)zeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Duplicate arguments are not always detected. Solution: Expand to full path before comparing arguments. (Nir Lichtman, closes vim/vim#11505, closes vim/vim#9402) https://github.com/vim/vim/commit/b3052aa1b555ab5a81b1459a4972290381b0e7e4 Co-authored-by: Nir Lichtman <nir@lichtman.org>
| * | | | | | | vim-patch:9.0.0863: col() and charcol() only work for the current window ↵zeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#21038) Problem: col() and charcol() only work for the current window. Solution: Add an optional winid argument. (Yegappan Lakshmanan, closes vim/vim#11466, closes vim/vim#11461) https://github.com/vim/vim/commit/4c8d2f02b3ce037bbe1d5ee12887e343c6bde88f Cherry-pick test_functions.vim change from patch 8.2.0633. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
| * | | | | | | vim-patch:9.0.0867: wildmenu redrawing code is spread out (#21035)zeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Wildmenu redrawing code is spread out. Solution: Refactor to move code together. (closes vim/vim#11528) https://github.com/vim/vim/commit/d6e91385f0f7256aec8f70373c9e3399770d22e5 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | | test(lua/ui_spec): fix Ctrl-C test flakiness (#21039)zeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | Prevent Ctrl-C from flushing the command that starts the prompt.
| * | | | | | | Merge pull request #21037 from zeertzjq/vim-8.2.4675zeertzjq2022-11-13
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | vim-patch:8.2.{4675,4676},9.0.0869: no error for missing expression after :elseif
| | * | | | | | | vim-patch:9.0.0869: bogus error when string used after :elseifzeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Bogus error when string used after :elseif. Solution: Do not consider a double quote the start of a comment. (closes vim/vim#11534) https://github.com/vim/vim/commit/28c56d501352bd98472d23667bade683877cadcc Co-authored-by: Bram Moolenaar <Bram@vim.org>
| | * | | | | | | vim-patch:8.2.4676: test fails with different errorzeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test fails with different error. Solution: Add argument for :elseif. https://github.com/vim/vim/commit/292e1b9f681054a1de8fa22315ae6eedd7acb205 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| | * | | | | | | vim-patch:8.2.4675: no error for missing expression after :elseifzeertzjq2022-11-13
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No error for missing expression after :elseif. (Ernie Rael) Solution: Check for missing expression. (closes vim/vim#10068) https://github.com/vim/vim/commit/fa010cdfb115fd2f6bae7ea6f6e63be906b5e347 Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | | vim-patch:9.0.0866: no test for what patch 8.2.2207 fixes (#21034)zeertzjq2022-11-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for what patch 8.2.2207 fixes. Solution: Add a test case. (closes vim/vim#11531) https://github.com/vim/vim/commit/f7570f2107d91f35dc67dd0e400fc638585b226c
| * | | | | | | vim-patch:9.0.0862: default value of 'endoffile' is wrong (#21032)zeertzjq2022-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Default value of 'endoffile' is wrong. Solution: The default must be 'noendoffile'. https://github.com/vim/vim/commit/0aad88f073602849d1623122eb3c323f8e252def Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | | fix: vim.ui.input always calls callback #21006Steven Arcangeli2022-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Followup to #20883 Related: #18144 This patch changes the behavior of the default `vim.ui.input` when the user aborts with `<C-c>`. Currently, it produces an error message + stack and causes `on_confirm` to not be called. With this patch, `<C-c>` will cause `on_confirm` to be called with `nil`, the same behavior as when the user aborts with `<Esc>`. I can think of three good reasons why the behavior should be this way: 1. Easier for the user to understand** It's not intuitive for there to be two ways to abort an input dialog that have _different_ outcomes. As a user, I would expect any action that cancels the input to leave me in the same state. As a plugin author, I see no value in having two possible outcomes for aborting the input. I have to handle both cases, but I can't think of a situation where I would want to treat one differently than the other. 2. Provides an API that can be overridden by other implementations** The current contract of "throw an error upon `<C-c>`" cannot be replicated by async implementations of `vim.ui.input`. If the callsite wants to handle the case of the user hitting `<C-c>` they need to use `pcall(vim.ui.input, ...)`, however an async implementation will instantly return and so there will be no way for it to produce the same error-throwing behavior when the user inputs `<C-c>`. This makes it impossible to be fully API-compatible with the built-in `vim.ui.input`. 3. Provides a useful guarantee to the callsite** As a plugin author, I want the guarantee that `on_confirm` will _always_ be called (only catastrophic errors should prevent this). If I am in the middle of some async thread of logic, I need some way to resume that logic after handing off control to `vim.ui.input`. The only way to handle the `<C-c>` case is with `pcall`, which as already mentioned, breaks down if you're using an alternative implementation.
| * | | | | | | Merge pull request #20178 from vigoux/extmark-nospellLewis Russell2022-11-12
| |\ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | | feat(extmarks): allow preventing spellchecking with spell = false
| | * | | | | | feat(spell): support nospell in treesitter queriesLewis Russell2022-11-12
| | | | | | | |
| | * | | | | | feat(extmarks): allow preventing spellchecking with spell = falseThomas Vigouroux2022-11-12
| |/ / / / / /
| * | | | | | vim-patch:9.0.0845: shell command with just space gives strange error (#21029)zeertzjq2022-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Shell command with just space gives strange error. Solution: Skip white space at start of the argument. (Christian Brabandt, Shane-XB-Qian, closes vim/vim#11515, closes vim/vim#11495) https://github.com/vim/vim/commit/4e7590ec00483077daaa567aa2220bc8df912f3c Co-authored-by: shane.xb.qian <shane.qian@foxmail.com>
| * | | | | | feat(checkhealth): use "help" syntax, avoid tabpage #20879Justin M. Keyes2022-11-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - If Nvim was just started, don't create a new tab. - Name the buffer "health://". - Use "help" syntax instead of "markdown". It fits better, and eliminates various workarounds. - Simplfy formatting, avoid visual noise. - Don't print a "INFO" status, it is noisy. - Drop the ":" after statuses, they are already UPPERCASE and highlighted.
| * | | | | | vim-patch:8.2.2207: illegal memory access if popup menu items are changed ↵zeertzjq2022-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#21028) Problem: Illegal memory access if popup menu items are changed while the menu is visible. (Tomáš Janoušek) Solution: Make a copy of the text. (closes vim/vim#7537) https://github.com/vim/vim/commit/38455a921395a56690790c8c1d28c1c43ca04c8a Co-authored-by: Bram Moolenaar <Bram@vim.org>
| * | | | | | vim-patch:9.0.0861: solution for "!!sort" in closed fold is not optimal (#21027)zeertzjq2022-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Solution for "!!sort" in closed fold is not optimal. Solution: Use a different range instead of the subtle difference in handling a range with an offset. (issue vim/vim#11487) https://github.com/vim/vim/commit/9954dc39ea090cee6bf41c888c41e60d9f52c3b8 Co-authored-by: Bram Moolenaar <Bram@vim.org>
generated by cgit (git 2.34.1) at 2025-06-24 00:10:17 +0000