aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #18255 from bfredl/workaround_versionbfredl2022-04-25
|\ | | | | fix(ci): workaround bug in msgpackparse() on windows caused by api level bump
| * fix(ci): workaround bug in msgpackparse() on windows caused by api level bumpbfredl2022-04-25
| |
* | fix: show autocmd output when F is in shortmess (#18251)Gregory Anders2022-04-25
|/ | | | | | | | | | | The default value of including F in 'shortmess' has the unfortunate side effect of hiding output from autocommands. This is a common source of confusion and often leads people to think their autocommands are not working when they are. There is a small snippet in the docs for 'shortmess' indicating that the F flag suppresses autocmd output, but it's not easy to find if you don't already know to look for it. This commit removes that behavior of the F flag to make it only suppress file info when opening a new file.
* revert: "man.vim: Ensure 'modifiable' in man#init_pager" #17791dundargoc2022-04-25
| | | | | | | | | | | | | This reverts commit 526798a941b4cf80fd1f128b40e51fb47c77b654. This will make man filetype not modifiable by default, as it is the superior behavior in my opinion. More importantly, also make it possible for a user to modify man filetypes by adding `set modifiable` in `~/.config/nvim/ftplugin/man.vim` or its equivalent. ref #11450 closes #17595 Co-authored-by: Javier López <graulopezjavier@gmail.com>
* Merge pull request #18234 from zeertzjq/cursearch-reduce-redrawzeertzjq2022-04-25
|\ | | | | perf: only redraw for CurSearch when it is currently in use
| * perf: only redraw for CurSearch when it is currently in usezeertzjq2022-04-25
| | | | | | | | | | | | The check in redraw_for_cursorline() is not needed because VALID_VIRTCOL is always cleared when the cursor moves to another line, so the check in redraw_for_cursorcolumn() is enough.
* | refactor: convert macros to all-caps (#17895)dundargoc2022-04-24
| | | | | | Closes https://github.com/neovim/neovim/issues/6297
* | refactor: enable Wconversion warning for diff (#18094)dundargoc2022-04-24
| | | | | | Work on https://github.com/neovim/neovim/issues/567
* | Merge pull request #18110 from dundargoc/refactor/remove-char_uGregory Anders2022-04-24
|\ \ | | | | | | refactor: replace char_u variables and functions with char
| * | refactor: add function xstrnsaveDundar Goc2022-04-16
| | | | | | | | | | | | | | | | | | | | | xstrnsave is a clone of vim_strnsave that uses char* instead of char_u*. Its purpose short-term is to help reduce the number of casts and for long-term to replace vim_strnsave as the need to use char_u is eliminated.
| * | refactor: replace char_u variables and functions with charDundar Göc2022-04-16
| | | | | | | | | | | | Work on https://github.com/neovim/neovim/issues/459
* | | refactor: add pure attribute to pure functions (#18165)dundargoc2022-04-24
| |/ |/| | | | | This will allow the compilers that support the pure attribute to make further optimizations.
* | fix/PVS #17863dundargoc2022-04-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix(PVS/V002): disable rule completely V002: "Some diagnostic messages may contain incorrect line number in this file." This particular check seems unreliable. It says on their website https://pvs-studio.com/en/docs/warnings/v002/ that this warning occurs when there are multiline pragmas, but there are none in extmark.c. * fix(PVS/V756): ignore "counter is not used inside a nested loop" warning The nested loop starts with "AutoCmd *ac = ap->cmds" so "ap" is definitely used. * fix(PVS/V560): disable "a part of conditional expression is always true" * fix(PVS/V614): potentially uninitialized variable 'blen' used
* | build: bump Doxyfile to minimum required version 1.9.0 #18118dundargoc2022-04-24
| |
* | docs: clarify enhanced modifiers, SHIFT usage #18124Joel Bradshaw2022-04-24
| | | | | | | | Co-authored-by: Justin M. Keyes <justinkz@gmail.com> Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* | ci: label all markdown edits as "documentation" #18120casswedson2022-04-24
| |
* | fix(packaging): remove excess forward slash in Wix Patch (#18121)sitiom2022-04-24
| |
* | Merge pull request #18248 from zeertzjq/vim-8.2.4806zeertzjq2022-04-25
|\ \ | | | | | | vim-patch:8.2.{4806,4818}: a mapping using <LeftDrag> does not start Select mode
| * | vim-patch:8.2.4818: no test for what 8.2.4806 fixeszeertzjq2022-04-25
| | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for what 8.2.4806 fixes. Solution: Add a test. (closes vim/vim#10727) https://github.com/vim/vim/commit/ac92ab771952b2a9ee39ea6fa5e70e4c072942d5 Test cannot be used because it must use test_setmouse(). Use a Lua test.
| * | vim-patch:8.2.4806: a mapping using <LeftDrag> does not start Select modezeertzjq2022-04-25
|/ / | | | | | | | | | | | | Problem: A mapping using <LeftDrag> does not start Select mode. Solution: When checking for starting select mode with the mouse also do this when there is typeahead. (closes vim/vim#10249) https://github.com/vim/vim/commit/53ef5731480d8b5aa74137a09b3b164b436ed76b
* | Merge pull request #18243 from zeertzjq/vim-8.2.4808zeertzjq2022-04-24
|\ \ | | | | | | vim-patch:8.2.{4808,4810,4813}: pasting text while indent folding may mess up folds
| * | vim-patch:8.2.4813: pasting text while indent folding may mess up foldszeertzjq2022-04-24
| | | | | | | | | | | | | | | | | | Problem: Pasting text while indent folding may mess up folds. Solution: Adjust the way folds are split. (Brandon Simmons, closes vim/vim#10254) https://github.com/vim/vim/commit/2c40707baa13a53cac4137ffb8b2ac67f50cea63
| * | vim-patch:8.2.4810: missing changes in one filezeertzjq2022-04-24
| | | | | | | | | | | | | | | | | | Problem: Missing changes in one file. Solution: Also change the struct initializers. https://github.com/vim/vim/commit/56dba60216a1bf72c1de299316f4d4ef19e50ad5
| * | vim-patch:8.2.4808: unused item in engine structzeertzjq2022-04-24
|/ / | | | | | | | | | | Problem: Unused item in engine struct. Solution: Remove "expr". Add comment with tags. https://github.com/vim/vim/commit/33d3ce640c63366e26b84c8d6f5798187a258ee2
* | build(clang-format): align with project style #18192dundargoc2022-04-23
| |
* | docs: make docstring consistent with parameters #18178dundargoc2022-04-23
| | | | | | Closes: https://github.com/neovim/neovim/issues/12691
* | ci(backport): remove triggering by comment (#18210)zeertzjq2022-04-23
| | | | | | | | Triggering by comment is not needed. Applying the label is enough to trigger the backport action.
* | Merge pull request #18197 from dundargoc/build/clint/python3Gregory Anders2022-04-23
|\ \ | | | | | | build/clint/python3
| * | build(clint): remove all python2-isms with pyupgradeDundar Goc2022-04-20
| | | | | | | | | | | | | | | The script is functionally the same, just with some nicer syntactic sugar thanks to only supporting python3.
| * | build(clint): change shebang to python3Dundar Goc2022-04-20
| | | | | | | | | | | | | | | | | | There are distributions that doesn't symlink python to python3. This will make clint work for these without any additional steps from the user.
* | | fix(ftdetect): source plugins in autogroup (#18237)Christian Clason2022-04-23
| | | | | | | | | In `filetype.lua`, source runtime `ftdetect` scripts within the `filetypedetect` augroup, same as `filetype.vim` (and only do so if `g:did_load_ftdetect` does not exist).
* | | Merge pull request #18226 from zeertzjq/vim-8.2.4805zeertzjq2022-04-23
|\ \ \ | | | | | | | | vim-patch:8.2.{4805,4812}: CurSearch used for all matches in current line
| * | | vim-patch:8.2.4812: unused struct itemzeertzjq2022-04-23
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Unused struct item. Solution: Remove "lines" match_T. Simplify the code. (closes vim/vim#10256) https://github.com/vim/vim/commit/8279cfe49961b3711c84c66a9954c9f70e9b78c8
| * | | vim-patch:8.2.4805: CurSearch used for all matches in current linezeertzjq2022-04-23
|/ / / | | | | | | | | | | | | | | | Problem: CurSearch used for all matches in current line. Solution: Don't use the non-zero line count. (closes vim/vim#10247) https://github.com/vim/vim/commit/9b36750640e8e89f18afa1446ed80fdbdf0fcac0
* | | fix(lua): don't mutate opts parameter of vim.keymap.del (#18227)Andrey Mishchenko2022-04-23
| | | | | | | | | | | | | | | | | | `vim.keymap.del` takes an `opts` parameter that lets caller refer to and delete buffer-local mappings. For some reason the implementation of `vim.keymap.del` mutates the table that is passed in, setting `opts.buffer` to `nil`. This is wrong and also undocumented.
* | | fix(treesitter): create new parser if language is not the same as cached ↵Chinmay Dalal2022-04-22
| | | | | | | | | | | | parser (#18149)
* | | Merge pull request #18211 from clason/update-inspectbfredl2022-04-22
|\ \ \ | | | | | | | | chore: update inspect.lua to 3.1.3
| * | | chore: update inspect.lua to 3.1.3Christian Clason2022-04-21
| | | |
* | | | docs(lua): explain and link to lua patterns (#18206)Christian Clason2022-04-21
| | | | | | | | | | | | also correct explanation of when it's allowed to omit parens in Lua function calls
* | | | refactor(normal): fix most clint warnings (#18196)dundargoc2022-04-21
| | | |
* | | | Merge pull request #18209 from zeertzjq/vim-8.2.4759zeertzjq2022-04-21
|\ \ \ \ | | | | | | | | | | vim-patch:8.2.4759: CurSearch highlight does not work for multi-line match
| * | | | vim-patch:8.2.4759: CurSearch highlight does not work for multi-line matchzeertzjq2022-04-21
|/ / / / | | | | | | | | | | | | | | | | | | | | Problem: CurSearch highlight does not work for multi-line match. Solution: Check cursor position before adjusting columns. (closes vim/vim#10133) https://github.com/vim/vim/commit/693ccd11606b59eb0f81c6c1948679e61ada4022
* | | | Merge pull request #18182 from zeertzjq/vim-8.2.2472zeertzjq2022-04-21
|\ \ \ \ | |/ / / |/| | | vim-patch:8.1.1756,8.2.{2472,2474,2475,2476,2477,4791,4802}: autocommand fixes
| * | | vim-patch:8.2.4802: test is not cleaned upzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test is not cleaned up. Solution: Make test clean up after itself. Avoid NUL. (closes vim/vim#10233) https://github.com/vim/vim/commit/7851c69a120ea6ce8c122dd7198adbe5aec83ea5 Adapt test_autocmd_vimgrep() to Nvim.
| * | | test: add a Lua functional test for NoName buffer event orderCharlie Groves2022-04-21
| | | |
| * | | vim-patch:8.2.4791: events triggered in different order when reusing bufferzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Autocmd events triggered in different order when reusing an empty buffer. Solution: Call buff_freeall() earlier. (Charlie Groves, closes vim/vim#10198) https://github.com/vim/vim/commit/fef4485ef58d5937b170c6dc69431359469fc9cd Test failure becomes very strange.
| * | | vim-patch:8.2.2477: autocommand tests hang on MS-Windowszeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Autocommand tests hang on MS-Windows. Solution: Skip a couple of tests. Fix file name. https://github.com/vim/vim/commit/dd07c02232e91ee963b91a4477179d4b9548b862
| * | | vim-patch:8.2.2476: using freed memory when splitting window while closing ↵zeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | buffer Problem: Using freed memory when using an autocommand to split a window while a buffer is being closed. Solution: Disallow splitting when the buffer has b_locked_split set. https://github.com/vim/vim/commit/983d83ff1cd796ff321074335fa53fbe7ac45a46 Put the error message in window.c. Cherry-pick a memory leak fix from Vim patch 8.2.0399. Test still fails.
| * | | vim-patch:8.2.2475: autocommand tests hangs on MS-Windowszeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Autocommand tests hangs on MS-Windows. Solution: Skip one test. https://github.com/vim/vim/commit/dfc3db76b9de217542cc9258301c1b4818a51cd0
| * | | vim-patch:8.2.2474: using freed memory when window is closed by autocommandzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Using freed memory when window is closed by autocommand. (houyunsong) Solution: Check the window pointer is still valid. https://github.com/vim/vim/commit/2c7080bf1ceef4a7779644fd428b2386a0676794 Add missing comment from Vim patch 8.0.1420. Test fails.