aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * vim-patch:8.2.3884: crash when clearing the argument list while using itzeertzjq2022-10-05
| | | | | | | | | | | | Problem: Crash when clearing the argument list while using it. Solution: Lock the argument list for ":all". https://github.com/vim/vim/commit/6f98371532fcff911b462d51bc64f2ce8a6ae682
| * vim-patch:8.2.2463: using :arglocal in an autocommand may use freed memoryzeertzjq2022-10-05
| | | | | | | | | | | | | | Problem: Using :arglocal in an autocommand may use freed memory. (houyunsong) Solution: Check if the arglist is locked. https://github.com/vim/vim/commit/6bcb877ec19a647443195a54eeac60cb693fd827
| * vim-patch:8.2.2421: double free when using autocommand with "argdel"zeertzjq2022-10-05
| | | | | | | | | | | | Problem: Double free when using autocommand with "argdel". (Houyunsong) Solution: Add the arglist_locked flag. https://github.com/vim/vim/commit/5ed58c7b700fcb9fd03c418300145b616f4bdcdd
| * vim-patch:8.2.2413: crash when using :all while using a cmdline windowzeertzjq2022-10-05
|/ | | | | | | | Problem: Crash when using :all while using a cmdline window. (Zdenek Dohnal) Solution: Disallow :all from the cmdline window. https://github.com/vim/vim/commit/bb4b93ed85726c3921596ca267f531c8c94d819a Use test from lastest Vim instead.
* Merge pull request #20494 from bfredl/shadatestbfredl2022-10-05
|\ | | | | test(shada): fix shada syntax definitions test.
| * test(shada): fix shada syntax definitions testbfredl2022-10-05
| | | | | | | | | | | | These tests contained errors due to synstack() and friends do not ensure syntax state is fully synced. Actually expecting what the user will see with a screen test does ensure it is fully synced.
* | feat(docs): format parameters as a list #20485Justin M. Keyes2022-10-05
| | | | | | | | | | | | | | | | | | | | | | Problem: The {foo} parameters listed in `:help api` and similar generated docs, are intended to be a "list" but they aren't prefixed with a list symbol. This prevents parsers from understanding the list, which forces generators like `gen_help_html.lua` to use hard-wrapped/preformatted layout instead of a soft-wrapped "flow" layout. Solution: Modify gen_vimdoc.py to prefix {foo} parameters with a "•" symbol.
* | Merge pull request #20493 from bfredl/windowtestbfredl2022-10-05
|\ \ | | | | | | test(api): migrate screenchar() test in in window API to screen test
| * | test(api): migrate screenchar() test in in window API to screen testbfredl2022-10-05
| |/ | | | | | | This produces actual output in case of regressions.
* | vim-patch:partial:f269eabc6c4f (#20470)Christian Clason2022-10-05
| | | | | | | | Update runtime files https://github.com/vim/vim/commit/f269eabc6c4f5bdcef989cd5b4b95ba8ccaa4d8a
* | Merge pull request #20490 from zeertzjq/vim-9.0.0656zeertzjq2022-10-05
|\ \ | |/ |/| vim-patch:9.0.{0656,0661}: 'fillchars' "lastline"
| * vim-patch:9.0.0661: multi-byte "lastline" in 'fillchars' does not work properlyzeertzjq2022-10-05
| | | | | | | | | | | | | | Problem: Multi-byte "lastline" item in 'fillchars' does not work properly when the window is two columns wide. Solution: Compute the text length correctly. (closes vim/vim#11280) https://github.com/vim/vim/commit/18b3500b8c517e44c23197e558aa36aed1c6916c
| * vim-patch:9.0.0656: cannot specify another character to use instead of '@'zeertzjq2022-10-05
|/ | | | | | | | | | Problem: Cannot specify another character to use instead of '@' at the end of the window. Solution: Add "lastline" to 'fillchars'. (Martin Tournoij, closes vim/vim#11264, closes vim/vim#10963) https://github.com/vim/vim/commit/4ba5f1dab656103e8f4a4505452d1816b9e83c1e Use latest code in drawscreen.c instead.
* fix(docs): missing "(" in :help HTMLJustin M. Keyes2022-10-04
| | | | | | | | | | | | | | | | | Problem: Since https://github.com/neovim/tree-sitter-vimdoc/commit/eba7b5b646546d9fed9b40b2c72b9cc0048f1dfa any opening paren and its leading whitespace " (" are missing in the generated HTML. Example: Use ":qa!<Enter>" (careful, all changes are lost!). ^^missing Position the cursor on a tag (e.g. bars) and hit CTRL-]. ^^missing Solution: The main recursive loop only processes named children, so check named_child_count() instead of child_count(). Then anonymous nodes won't get lost.
* feat(lsp): add bufnr option to lsp.start (#20473)August Masquelier2022-10-04
|
* fix(build): "make clean" failsJustin M. Keyes2022-10-04
| | | | | | | | | | | | Problem: "make clean" fails since 03bc23de36c8. make -C runtime/doc clean make[1]: *** No rule to make target `clean'. Stop. make: *** [clean] Error 2 Solution: Update the "clean" target.
* Merge #20412 feat(docs): HTML generatorJustin M. Keyes2022-10-04
|\
| * feat(gen_help_html.lua): remove old AWK scriptsJustin M. Keyes2022-10-04
| | | | | | | | | | | | | | These files are no longer needed since gen_help_html.lua is working fairly well. ref https://github.com/neovim/neovim/pull/11967
| * feat(docs): nested lists in HTML, update :help parserJustin M. Keyes2022-10-04
|/ | | | | | | | | | | | - Docs HTML: improvements in https://github.com/neovim/tree-sitter-vimdoc allow us to many hacks in `gen_help_html.lua`. - Docs HTML: support nested lists. - Docs HTML: avoid extra newlines (too much whitespace) in old (preformatted) layout. - Docs HTML: disable golden-grid for narrow viewport. - Workaround for https://github.com/neovim/neovim/issues/20404 closes https://github.com/neovim/neovim/issues/20404
* fix(ui): msg_ext_set_kind for nvim_echo (#20476)Shougo2022-10-04
|
* build(deps): bump LuaJIT to HEAD - 6c4826f12 (#20478)Christian Clason2022-10-04
|
* Merge pull request #20467 from zeertzjq/vim-8.2.2316zeertzjq2022-10-04
|\ | | | | vim-patch:8.2.2316: Vim9: cannot list a lambda function
| * test: add test for :function followed by <lambda>zeertzjq2022-10-04
| |
| * vim-patch:8.2.2316: Vim9: cannot list a lambda functionzeertzjq2022-10-04
|/ | | | | | Problem: Vim9: cannot list a lambda function. Solution: Support the <lambda>9 notation, like :disassemble. (closes vim/vim#7634) https://github.com/vim/vim/commit/b657198cb30765468451d7f68fce49b5b4000c5d
* docs(shell): mention "&" for piping with powershell #20459Enan Ajmain2022-10-03
| | | New behaviour since PR #19438
* Merge #20154 build: cmake cleanupJustin M. Keyes2022-10-02
|\
| * build: define EP_PREFIX propertydundargoc2022-10-02
| | | | | | | | | | This is just to avoid the boilerplate of definining PREFIX for each dependency.
| * build: rely on builtin cmake downloading rather than custom scriptdundargoc2022-10-02
| | | | | | | | | | | | | | DownloadAndExtractFile.cmake was initially introduced as a workaround to avoid the massive amounts of logs generated by the download progress. This is not a problem anymore as ExternalProject_Add has had the DOWNLOAD_NO_PROGRESS option since cmake version 3.1.
| * build: remove unnecessary translation-related codedundargoc2022-10-02
| | | | | | | | | | The commands run in cmake script mode (-P) can simply be run in the main cmake run instead.
| * build: remove url for 32-bit winyankdundargoc2022-10-02
| | | | | | | | | | | | We don't support 32bit Windows anymore, so it's not needed. Also remove TargetArch.cmake and related code as we don't need architecture detection for the same reason.
| * build: remove code for cross-compilationdundargoc2022-10-02
| | | | | | | | | | We don't support cross-compilation at the moment, so these can be safely removed.
| * build: remove unused variable CMAKE_C_COMPILER_ARG1dundargoc2022-10-02
| | | | | | | | | | It was set in file cmake/i386-linux-gnu.toolchain.cmake which has been removed since we don't use Travis anymore.
* | docs: refer to vim.lsp.start() in LSP issue template #20422Mathias Fußenegger2022-10-02
| | | | | | The nvim-lspconfig minimal_init.lua is not exactly minimal.
* | feat(l10n): update Turkish translations (#20444)Emir SARI2022-10-02
| |
* | build: only generate compilation database for the nvim target (#20449)dundargoc2022-10-02
|/ | | | | | | | | | | Some tools like clang-tidy get confused and repeat unnecessary work if there are multiple entries with the same name. This will only generate a compilation database for cmake version 3.20 and above. Generating a compilation database is only necessary for development, so we don't need to maintain compatibility with the minimum required version. Closes https://github.com/neovim/neovim/issues/10632
* build(deps): bump Luv to HEAD - 80c8c00ba (#20433)Christian Clason2022-10-02
|
* build(deps): bump libuv to HEAD - f610339f7 (#20445)Christian Clason2022-10-02
| | | also remove libuv-disable-shared patch
* fix(folds): fix fold marker multibyte comparison (#20439)zeertzjq2022-10-02
|
* Merge pull request #20429 from zeertzjq/vim-9.0.0620zeertzjq2022-10-02
|\ | | | | vim-patch:9.0.{0620,0622}: matchaddpos() can only add up to 8 matches
| * vim-patch:9.0.0622: matchaddpos() can get slow when adding many matcheszeertzjq2022-10-02
| | | | | | | | | | | | | | Problem: matchaddpos() can get slow when adding many matches. Solution: Update the next available match ID when manually picking an ID and remove check if the available ID can be used. (idea by Rick Howe) https://github.com/vim/vim/commit/9f573a8df02d9f699a43d2afbd1d2841d700b9ad
| * vim-patch:9.0.0620: matchaddpos() can only add up to 8 matcheszeertzjq2022-10-02
|/ | | | | | Problem: matchaddpos() can only add up to 8 matches. Solution: Allocate the array of positions. (closes vim/vim#11248) https://github.com/vim/vim/commit/50faf02f43d7f1a56ec2023028fca7c72dbce83e
* Merge #19438 from 3N4N/fix/pwshJustin M. Keyes2022-10-01
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reverts #16271 Fixs #15913 Problem: Since #16271, `make_filter_cmd` uses `Start-Process` cmdlet to execute the user provided shell command for `:%!`. `Start-Process` requires the command to be split into the shell command and its arguments. This was implemented in #19268 by parsing (splitting the user-provided command at the first space) which didn't handle cases such as -- - commands with escaped space in their filepath - quoted commands with space in their filepath Solution: Use piping. The total shell command formats (excluding noise of unimportant parameters): 1. Before #16271 ```powershell pwsh -C "(shell_cmd) < tmp.in | 2>&1 Out-File -Encoding UTF8 <tmp.out>" # not how powershell commands work ``` 2. Since #16271 ```powershell pwsh -C "Start-Process shell_cmd -RedirectStandardInput <tmp.in> -RedirectStandardOutput <tmp.out>" # doesn't handle executable path with space in it # doesn't write error to <tmp.out> ``` 3. This PR ```powershell pwsh -C "& { Get-Content <tmp.in> | & 'path\with space\to\shell_cmd.exe' arg1 arg2 } 2>&1 | Out-File -Encoding UTF8 <tmp.out>" # also works with forward slash in the filepath # also works with double quotes around shell command ``` After this PR, the user can use the following formats: :%!c:\Program` Files\Git\usr\bin\sort.exe :%!'c:\Program Files\Git\usr\bin\sort.exe' :%!"c:\Program Files\Git\usr\bin\sort.exe" :%!"c:\Program` Files\Git\usr\bin\sort.exe" They can even chain different commands: :%!"c:\Program` Files\Git\usr\bin\sort.exe" | sort.exe -r But if they want to call a stringed executable path, they have to provide the Invoke-Command operator (&). In fact, the first stringed executable path also needs this & operator, but this PR adds that behind the scene. :%!"c:\Program` Files\Git\usr\bin\sort.exe" | sort.exe -r | & 'c:\Program Files\Git\usr\bin\sort.exe' ## What this PR solves - Having to parse the user-provided bang ex-command (for splitting into shell cmd and its args). - Removes a lot of human-unreadable `#ifdef` blocks. - Accepting escaped spaces in executable path. - Accepting quoted string of executable path. - Redirects error and exception to tmp.out (exception for when `wrong_cmd.exe not found`) ## What this PR doesn't solve - Handling wrongly escaped path to executable, which the user may pass because of cmdline tab-completion. #18592 ## Edge cases - (Not handled) If the user themself provides the `&` sign (means `call this.exe` in powershell) - (Not handled) Use `-Encoding utf8` parameter for `Get-Content`? - (Handled) Doesn't write to tmp.out if shell command is not found. - fix: use anonymous function (`{wrong_cmd.exe}`). ## Changes other than `make_filter_cmd()` function - Encoding for piping to external executables. See BOM-less UTF8: https://github.com/PowerShell/PowerShell/issues/4681
| * fix: :! pwsh redirection for `command not found`Enan Ajmain2022-09-30
| | | | | | | | | | | | | | | | | | Problem: If the shell command passed to the filtered bang command isn't found, the error isn't redirected to the temp.out file when shell is set to powershell. Solution: Use anonymous function with Invoke-Command operator (&).
| * fix: make_filter_cmd for :! powershellEnan Ajmain2022-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: `Start-Process` requires the command to be split into the shell command and its arguments. Previously it was done by parsing, which didn't handle cases such as - commands with escaped space in their filepath - quoted commands with space in their filepath Solution: Use - `pwsh -Command` instead of `Start-Process` - `Get-Content` instead of `-RedirectStandardInput` - `Out-File` instead of `-RedirectStandardOutput`
* | ci(publish-winget): use versioned tags for action #20417Vedant2022-10-01
| |
* | refactor(diagnostic): remove deprecated function (#20423)Raphael2022-10-01
| |
* | vim-patch:8.2.2542: highlight of char beyond line end is not correct (#20424)zeertzjq2022-10-01
| | | | | | | | | | | | | | | | Problem: Highlight of char beyond line end is not correct. (Chuan Wei Foo) Solution: Fix counting NUL as one cell. Draw one more character if the EOL is part of the match. (closes vim/vim#7883) https://github.com/vim/vim/commit/41f0895c6e3c7b921e3c102ad42be52b1be48018 Reorder test_search.vim to match Vim.
* | refactor(lsp): remove deprecated lsp functions (#20421)Mathias Fußenegger2022-10-01
| |
* | Merge pull request #20414 from dundargoc/ci/release/matrixJames McCoy2022-09-30
|\ \ | | | | | | ci(release): remove unnecessary matrix strategy
| * | ci(release): remove unnecessary matrix strategydundargoc2022-09-30
| |/ | | | | | | We only have one Windows release job, so the matrix setup isn't needed.