aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | test(old): add test_cdo.vim and test_packadd.vimzeertzjq2022-02-13
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | test_cdo.vim is copied from Vim v8.1.1483. test_packadd.vim is copied from Vim v8.2.0174.
* | | | | | Merge pull request #17388 from zeertzjq/test-assertzeertzjq2022-02-13
|\ \ \ \ \ \ | | | | | | | | | | | | | | test(old): move some tests from assert_spec.lua to test_assert.vim
| * | | | | | test(old): move some tests from assert_spec.lua to test_assert.vimzeertzjq2022-02-13
|/ / / / / /
* | | | | | Merge pull request #17385 from seandewar/vim-8.2.4359Sean Dewar2022-02-12
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch:8.2.{4359,4362,4363,4364}: crash when repeatedly using :retab
| * | | | | | vim-patch:8.2.4364: MS-Windows: still running out of memory for a very long lineSean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: MS-Windows: still running out of memory for a very long line. Solution: Check for negative length. https://github.com/vim/vim/commit/45491660787043ea412719544881db691338d730
| * | | | | | vim-patch:8.2.4363: MS-Windows: running out of memory for a very long lineSean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: MS-Windows: running out of memory for a very long line. Solution: Use a 32 bit value for MAXCOL also when ints are 64 bits. https://github.com/vim/vim/commit/8e38555ece7d3fe1edc6681ec70fe5586a524862 This still fails Vim's Windows CI, so let's see what happens...
| * | | | | | vim-patch:8.2.4362: :retab may allocate too much memorySean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: :retab may allocate too much memory. Solution: Bail out when allocating more than MAXCOL bytes. https://github.com/vim/vim/commit/33f3c5985491032d5bdfc30e722e85d5a0285e64
| * | | | | | vim-patch:8.2.4359: crash when repeatedly using :retabSean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: crash when repeatedly using :retab. Solution: Bail out when the line is getting too long. https://github.com/vim/vim/commit/6e28703a8e41f775f64e442c5d11ce1ff599aa3f Cherry-pick e_resulting_text_too_long from v8.2.3492; put it in globals.h as it will eventually be used in other files. Add a modeline to test_retab.vim
* | | | | | | Merge pull request #16643 from zeertzjq/vim-8.1.2184zeertzjq2022-02-13
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:8.1.2184,8.2.3804: option context is not copied when splitting a window
| * | | | | | | vim-patch:8.1.2184: option context is not copied when splitting a windowzeertzjq2022-02-13
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Option context is not copied when splitting a window. (Daniel Hahler) Solution: Copy the option context, so that ":verbose set" works. (closes vim/vim#5066) https://github.com/vim/vim/commit/cfb381421f8be7d6cb4e7dac5b827b23467d3e53 vim-patch:8.2.3804: script context not set when copying 'swf' and 'ts' Problem: Script context not set when copying 'swf' and 'ts'. Solution: Use COPY_OPT_SCTX with the right argument. (closes vim/vim#9347) https://github.com/vim/vim/commit/6206877c511c636cbeb2a2b911451af316b62d00
* | | | | | | Merge pull request #17382 from zeertzjq/vim-8.2.2342zeertzjq2022-02-13
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | vim-patch:8.2.2342: "char" functions may return wrong column in Insert mode
| * | | | | | vim-patch:8.2.2342: "char" functions may return wrong column in Insert modezeertzjq2022-02-12
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: "char" functions return the wront column in Insert mode when the cursor is beyond the end of the line. Solution: Compute the column correctly. (Yegappan Lakshmanan, closes vim/vim#7669) https://github.com/vim/vim/commit/9145846b6aa411e3ab5c0d145b37808654352877
* | | | | | Merge pull request #17379 from clason/vim-8.2.4352Christian Clason2022-02-12
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | vim-patch:8.2.4352: ReScript files are not recognized
| * | | | | vim-patch:8.2.4352: ReScript files are not recognizedChristian Clason2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ReScript files are not recognized. Solution: Add the *.res and *.resi patterns. (Ananda Umamil, closes vim/vim#9752) https://github.com/vim/vim/commit/0c3cc2fec31521b0697edc406f85b7a43e979860
* | | | | | Merge pull request #17383 from lewis6991/sign_effChristian Clason2022-02-12
|\ \ \ \ \ \ | | | | | | | | | | | | | | refactor(signs): more efficient signcol calc
| * | | | | | refactor(signs): more efficient signcol calcLewis Russell2022-02-12
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | When iterating signs to calculate the sign column, stop iterating when we reach the maximum configured from 'signcolumn'.
* | | | | | Merge pull request #17381 from zeertzjq/vim-8.2.0580zeertzjq2022-02-12
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | vim-patch:8.2.0580: window size wrong if 'ea' is off and 'splitright' is on
| * | | | | vim-patch:8.2.0580: window size wrong if 'ea' is off and 'splitright' is onzeertzjq2022-02-12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Window size wrong if 'ea' is off and 'splitright' is on and splitting then closing a window. Solution: Put abandoned window space in the right place. (Mark Waggoner) https://github.com/vim/vim/commit/edd327cc070d9a05c12e88bc5c43a1e2a3086ae6
* | | | | Merge pull request #17377 from clason/keyset-morebfredl2022-02-12
|\ \ \ \ \ | | | | | | | | | | | | feat(highlight): add strikethrough, nocombine to `nvim_set_hl`
| * | | | | feat(api): add strikethrough, nocombine to set_hlChristian Clason2022-02-12
| | | | | |
* | | | | | Merge pull request #17269 from seandewar/vim-8.2.0915Sean Dewar2022-02-12
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch:8.2.{0915,0918,0922,4355}
| * | | | | | vim-patch:8.2.0922: search test failsSean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Search test fails. Solution: Remove failure tests for calls that no longer fail. https://github.com/vim/vim/commit/48af321a3382008dc642362d3f54bb6a61ff36e4
| * | | | | | vim-patch:8.2.0918: duplicate code for evaluating expression argumentSean Dewar2022-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Duplicate code for evaluating expression argument. Solution: Merge the code and make the use more flexible. https://github.com/vim/vim/commit/a9c010494767e43a51c443cac35ebc80d0831d0b
| * | | | | | vim-patch:8.2.0915: search() cannot skip over matches like searchpair() canSean Dewar2022-02-12
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Search() cannot skip over matches like searchpair() can. Solution: Add an optional "skip" argument. (Christian Brabandt, closes vim/vim#861) https://github.com/vim/vim/commit/adc17a5f9d207fd1623fd923457a46efc9214777 Enable skip arg usage in autoload/freebasic.vim evalarg_T doesn't really matter because it's deleted in v8.2.0918 (and reincarnated for Vim9 script in v8.2.1047), but I found out too late :P Anyway: - Port evalarg_T into eval.h and use const char * and Callback fields - Use EVALARG_INIT to initialize - Return bool over OK/FAIL from evalarg functions - Remove check from evalarg_clean as callback_free ignores None callbacks anyway - Move eva_buf field into evalarg_get as a local (not sure what reason it has being in the struct) N/A patches for version.c: vim-patch:8.2.4355: unnecessary call to check_colorcolumn() Problem: Unnecessary call to check_colorcolumn(). Solution: Remove the call. (Sean Dewar, closes vim/vim#9748) https://github.com/vim/vim/commit/0f7ff851cb721bb3c07261adbf82b591229f530d
* | | | | | Merge pull request #17376 from dundargoc/ci/reviews/upgrade-to-v6James McCoy2022-02-12
|\ \ \ \ \ \ | | | | | | | | | | | | | | ci(reviews): upgrade github-script version to v6
| * | | | | | ci(reviews): upgrade github-script version to v6Dundar Göc2022-02-12
| |/ / / / /
* | | | | | Merge pull request #17371 from dundargoc/revert/continue-on-errorJames McCoy2022-02-12
|\ \ \ \ \ \ | |/ / / / / |/| | | | | revert: "ci: use continue-on-error instead of "|| true""
| * | | | | revert: "ci: use continue-on-error instead of "|| true""Dundar Göc2022-02-12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 559aa4179c639f5ebd45474a36361343ac2371cf. I mistakenly believed both ways of writing were equivalent; this is untrue. Setting continue-on-error to true will make the job pass, but an error annotation will still be created which is misleading since it's not actually an error.
* | | | | Merge pull request #17370 from dundargoc/ci/fix-reviews-workflowJames McCoy2022-02-11
|\ \ \ \ \ | |/ / / / |/| | | | ci: fix reviews workflow by sending an artifact with PR number
| * | | | ci: fix reviews workflow by sending an artifact with PR numberDundar Göc2022-02-11
| | | | |
* | | | | Merge pull request #16841 from zeertzjq/vim-8.1.2073zeertzjq2022-02-12
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.1.2073: when editing a buffer 'colorcolumn' may not work
| * | | | | test: convert some colorcolumn tests to Lua screen testszeertzjq2022-02-12
| | | | | |
| * | | | | vim-patch:8.1.2073: when editing a buffer 'colorcolumn' may not workzeertzjq2022-02-12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When editing a buffer 'colorcolumn' may not work. Solution: Set the buffer before copying option values. Call check_colorcolumn() after copying window options. https://github.com/vim/vim/commit/010ee9657acf1a9f799079d718998c94e50ccadc
* | | | | Merge pull request #17325 from glacambre/stdioopen_docJames McCoy2022-02-11
|\ \ \ \ \
| * | | | | docs(stdioopen): add missing documentation for on_print paramglacambre2022-02-11
| | | | | | | | | | | | | | | | | | | | | | | | This commit adds documentation for the feature introduced in #15910.
* | | | | | Merge pull request #17372 from dundargoc/ci/commitlint/fix-revert-bugJames McCoy2022-02-11
|\ \ \ \ \ \
| * | | | | | ci(commitlint): allow first non-space character to be a quoteDundar Göc2022-02-11
|/ / / / / /
* | | | | | Merge pull request #17284 from dundargoc/ci/reduce-noiseJames McCoy2022-02-11
|\ \ \ \ \ \
| * | | | | | ci: disable tracing (set -x) from the shell scriptsDundar Göc2022-02-11
|/ / / / / /
* | | | | | Merge pull request #17365 from kevinhwang91/fix-ts-empty-linesChristian Clason2022-02-11
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | fix(query.lua): check empty table for lines
| * | | | | fix: append test for checking zero width node rangekevinhwang912022-02-11
| | | | | |
| * | | | | fix(query.lua): check empty table for lineskevinhwang912022-02-11
| | |_|/ / | |/| | | | | | | | | | | | | The range of node may make `nvim_buf_get_lines` return an empty table.
* | | | | fix(lsp): correct prefix when filterText is present (#17051)Lajos Koszti2022-02-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | LSP server might return an item which would replace a token to another. For example in typescript for a `jest.Mock` object `getProductsMock.` text I get the following response: ``` { commitCharacters = { ".", ",", "(" }, data = { entryNames = { "Symbol" }, file = "/foo/bar/baz.service.spec.ts", line = 268, offset = 17 }, filterText = ".Symbol", kind = 6, label = "Symbol", sortText = "11", textEdit = { newText = "[Symbol]", range = { end = { character = 16, line = 267 }, start = { character = 15, line = 267 } } } }, ``` In `lsp.omnifunc` to get a `prefix` we call the `adjust_start_col` which then returns the `textEdit.range.start.character`. Th `prefix` then be the `.` character. Then when filter the items with `remove_unmatch_completion_items`, every item will be filtered out, since no completion word starts `.`. To fix we return the `end.character`, which in that particular case will be the position after the `.`.
* | | | | Merge pull request #17368 from zeertzjq/test-filechanged-dialogzeertzjq2022-02-11
|\ \ \ \ \ | | | | | | | | | | | | test: convert Test_file_changed_dialog() to Lua functional test
| * | | | | test: convert Test_file_changed_dialog() to Lua functional testzeertzjq2022-02-11
|/ / / / /
* | | | | Merge pull request #17360 from lewis6991/fix-window-closeChristian Clason2022-02-11
|\ \ \ \ \ | |/ / / / |/| | | | fix: close floating windows when calling win_close()
| * | | | fix: close floating windows when calling win_close()Rom Grk2022-02-10
| | | | |
* | | | | Merge pull request #17012 from EdmundsEcho/fix-checkhealthzeertzjq2022-02-11
|\ \ \ \ \ | | | | | | | | | | | | prevent checkhealth failure when plugin's check returns void
| * | | | | fix(healthcheck): handle empty reportsEdmund Cape2022-02-09
| | | | | |
* | | | | | Merge pull request #17353 from zeertzjq/api-changedir-funczeertzjq2022-02-11
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix(api): use changedir_func() in nvim_set_current_dir()