aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
* 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.
* 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
| |
* | 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
* | 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
* | docs: make docstring consistent with parameters #18178dundargoc2022-04-23
| | | | | | Closes: https://github.com/neovim/neovim/issues/12691
* | 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.
* | | 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
* | | refactor(normal): fix most clint warnings (#18196)dundargoc2022-04-21
| | |
* | | 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
* | | 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.
* | | 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.
* | | vim-patch:8.2.2472: crash when using command line window in an autocommandzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when using command line window in an autocommand. (houyunsong) Solution: Save and restore au_new_curbuf. https://github.com/vim/vim/commit/aad5f9d79a2b71e9d2581eace3652be156102b9d Nvim has removed :open, so use :edit in the test instead.
* | | vim-patch:8.1.1756: autocommand that splits window messes up window layoutzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Autocommand that splits window messes up window layout. Solution: Disallow splitting a window while closing one. In ":all" give an error when moving a window will not work. https://github.com/vim/vim/commit/1417c766f55e5959b31da488417b7d9b141404af Expected error number was changed to E242 in Vim in patch 8.2.1183, and patch 8.2.2420 (which has already been ported) made the test no longer throw E249 in Vim, so just use E242 in the test.
* | | Merge pull request #18199 from zeertzjq/vim-8.2.4797zeertzjq2022-04-21
|\ \ \ | | | | | | | | vim-patch:8.2.4797: getwininfo() may get oudated values
| * | | vim-patch:8.2.4797: getwininfo() may get oudated valueszeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: getwininfo() may get oudated values. Solution: Make sure w_botline is up-to-date. (closes vim/vim#10226) https://github.com/vim/vim/commit/8530b41fd3872c9a1349b083470d565677948518 Correct test order and add a modeline in test_bufwintabinfo.vim.
* | | | vim-patch:8.2.4794: compiler warning for not initialized variablezeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Compiler warning for not initialized variable. Solution: Initialize the variable. (John Marriott) https://github.com/vim/vim/commit/4c84dd33ad739237bb38bb4e51702af972b5e507
* | | | vim-patch:8.2.4792: indent operator creates an undo entry for every linezeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Indent operator creates an undo entry for every line. Solution: Create one undo entry for all lines. (closes vim/vim#10227) https://github.com/vim/vim/commit/e4686989944bffdbcf59890aa21091b135528618
* | | | vim-patch:8.2.0358: insufficient testing for indent.czeertzjq2022-04-21
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insufficient testing for indent.c. Solution: Add indent tests. (Yegappan Lakshmanan, closes vim/vim#5736) https://github.com/vim/vim/commit/bd7206e02c957f0619e68e1628e2a3e91dd41e06 Cherry-pick Test_ex_mode() changes from Vim patches 8.2.{0342,0347}. Reorder test_expand_func.vim to match upstream.
* | | Merge pull request #18189 from zeertzjq/vim-8.2.4795zeertzjq2022-04-21
|\ \ \ | | | | | | | | vim-patch:8.2.{4795,4796,4801}: 'cursorbind' scrolling depends on whether 'cursorline' is set
| * | | vim-patch:8.2.4801: fix for cursorbind fix not fully testedzeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Fix for cursorbind fix not fully tested. Solution: Add another test case. (Christian Brabandt, closes vim/vim#10240) https://github.com/vim/vim/commit/3fd7480cd25f1e939fc2362e0644d497bcc81b71
| * | | vim-patch:8.2.4796: file left behind after running cursorline testszeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | Problem: File left behind after running cursorline tests. Solution: Uncomment the line that deletes the file. https://github.com/vim/vim/commit/da1050cd6fbb67cfde5b4a149d8d9db80bb4351c
| * | | vim-patch:8.2.4795: 'cursorbind' scrolling depends on whether 'cursorline' ↵zeertzjq2022-04-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | is set Problem: 'cursorbind' scrolling depends on whether 'cursorline' is set. Solution: Always call validate_cursor(). (Christian Brabandt, closes vim/vim#10230, closes vim/vim#10014) https://github.com/vim/vim/commit/2c645e8b00641f504072f35e061b7392ed41f491
* | | | vim-patch:8.2.4724: current instance of last search pattern not easily spottedzeertzjq2022-04-21
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Current instance of last search pattern not easily spotted. Solution: Add CurSearch highlighting. (closes vim/vim#10133) https://github.com/vim/vim/commit/a43993897aa372159f682df37562f159994dc85c This fixes CurSearch highlight for multiline match. Omit screen redrawing code because Nvim redraws CurSearch differently.
* | | Merge pull request #18185 from clason/vim-8.2.4781Christian Clason2022-04-20
|\ \ \ | |/ / |/| | vim-patch:8.2.{4781,4793}: maxima files are not recognized
| * | vim-patch:8.2.4793: recognizing Maxima filetype even though it might be anotherChristian Clason2022-04-20
| | | | | | | | | | | | | | | | | | Problem: Recognizing Maxima filetype even though it might be another. Solution: Remove *.mc and *.dem patterns from Maxima files https://github.com/vim/vim/commit/928a13135629fa8e73796760077b1b877918a080
| * | vim-patch:8.2.4781: Maxima files are not recognizedChristian Clason2022-04-20
| | | | | | | | | | | | | | | | | | Problem: Maxima files are not recognized. Solution: Add patterns to detect Maxima files. (Doron Behar, closes vim/vim#10211) https://github.com/vim/vim/commit/d0a20c9d111da75febb60ffee2e15f727ab6a5ad
* | | docs(api): add example showing necessity to wrap callback function (#18179)Mathias Fußenegger2022-04-20
| | | | | | | | | | | | | | | | | | | | | Some people ran into issues trying to use `callback = myluafun` because of the event data payload. Co-authored-by: Gregory Anders <8965202+gpanders@users.noreply.github.com>
* | | Merge pull request #18145 from bfredl/term_optbfredl2022-04-20
|\ \ \ | |/ / |/| | feat(api): allow remote UI to set terminal options
| * | feat(api): ui options relevant for remote TUIhlpr982022-04-17
| | |
* | | vim-patch:8.2.4790: lilypond filetype not recognized (#18174)Christian Clason2022-04-20
| | | | | | | | | | | | | | | Problem: Lilypond filetype not recognized. Solution: Add patterns for lilypond. (Doug Kearns) https://github.com/vim/vim/commit/c448e9c95089b5e7170a0dd36635a226c5eb5b1c
* | | Merge pull request #18157 from famiu/feat/undo!bfredl2022-04-19
|\ \ \ | | | | | | | | feat: add `undo!`