aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | vim-patch:8.2.3882: more duplicated code in f_getreginfo()zeertzjq2021-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: More duplicated code in f_getreginfo(). Solution: Also use getreg_get_regname(). (closes vim/vim#9398) https://github.com/vim/vim/commit/d3f00f54bf955bd01767db3a0af25866bc112ec7
| * | | | | | vim-patch:8.2.3879: getreg() and getregtype() contain dead codezeertzjq2021-12-24
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: getreg() and getregtype() contain dead code. Solution: Remove the needless check. (closes vim/vim#9392) Also refactor to put common code in a shared function. https://github.com/vim/vim/commit/51e64b2789eb7e60f7c5892a43426ab4ec1a54aa
* | | | | | Merge pull request #16705 from zeertzjq/vim-8.2.2508Jan Edmund Lazo2021-12-25
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch:8.2.{2508,2641,3846}
| * | | | | | vim-patch:8.2.3846: no error when using control character for 'lcs' or 'fcs'zeertzjq2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No error when using control character for 'lcs' or 'fcs'. Solution: Use char2cells() to check the width. (closes vim/vim#9369) https://github.com/vim/vim/commit/60618c8f1a7ea55452837a446525272142286471
| * | | | | | vim-patch:8.2.2641: display test fails because of lacking redrawzeertzjq2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Display test fails because of lacking redraw. Solution: Add a redraw command. https://github.com/vim/vim/commit/2cec027af461095f96dec3bfd036c267f790b0f4
| * | | | | | vim-patch:8.2.2508: cannot change the character displayed in non existing lineszeertzjq2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot change the character displayed in non existing lines. Solution: Add the "eob" item to 'fillchars'. (closes vim/vim#7832, closes vim/vim#3820) https://github.com/vim/vim/commit/a98f8a230596d8fb44cc68321de72980a21428cb Nvim has already implemented this feature, so this just ports the tests and docs.
* | | | | | | vim-patch:8.2.3883: crash when switching to other regexp engine fails (#16773)zeertzjq2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when switching to other regexp engine fails. Solution: Check for regprog being NULL. https://github.com/vim/vim/commit/5937c7505f444dd896f336fa0119a93a55ebe9a2
* | | | | | | Merge pull request #16789 from janlazo/vim-8.1.1184Jan Edmund Lazo2021-12-25
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:8.1.{1184,1379}
| * | | | | | | vim-patch:8.1.1379: filechanged test hangsJan Edmund Lazo2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Filechanged test hangs. Solution: Do not check 'autoread'. https://github.com/vim/vim/commit/8239c62067c8d40720560496b25a82662126f2a2
| * | | | | | | vim-patch:8.1.1184: undo file left behind after running testJan Edmund Lazo2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Undo file left behind after running test. Solution: Delete the undo file. (Dominique Pelle, closes vim/vim#4279) https://github.com/vim/vim/commit/137c14bb4f18198ed38659dcfbdfd749115c7ab5
| * | | | | | | test: port test_filechanged.vimJan Edmund Lazo2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I can't port patch 8.1.0820. Attempt failed in https://github.com/neovim/neovim/pull/12948. test_filechanged.vim is required for later patches.
* | | | | | | | vim-patch:8.2.3880: Solution filter files are not recognized (#16779)Christian Clason2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Solution filter files are not recognized. Solution: Add pattern *.slnf and use json. (Doug Kearns) https://github.com/vim/vim/commit/70a120b72b2b48800f881725c58241cd170893f8
* | | | | | | | Merge pull request #16724 from VVKot/vim-8.2.3850Jan Edmund Lazo2021-12-25
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | vim-patch:8.2.{3850,3855}
| * | | | | | | vim-patch:8.2.3855: illegal memory access when displaying a blobVVKot2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Illegal memory access when displaying a blob. Solution: Append a NUL at the end. (Yegappan Lakshmanan, closes vim/vim#9372) https://github.com/vim/vim/commit/bc404bfb32cf2bef34050d2aeae0ea72ccf980cc
| * | | | | | | vim-patch:8.2.3850: illegal memory access when displaying a partialVVKot2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Illegal memory access when displaying a partial. Solution: Terminate the string with a NUL. (closes vim/vim#9371) https://github.com/vim/vim/commit/2de5371a755abd287dab6ff544924715a76d4abe
* | | | | | | | test(old): reenable Test_virtual_replace() (#16674)zeertzjq2021-12-25
| | | | | | | |
* | | | | | | | feat(lua): add vim.spell (#16620)Lewis Russell2021-12-25
| | | | | | | |
* | | | | | | | Merge pull request #16725 from VVKot/vim-8.1.0040Björn Linse2021-12-25
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | | vim-patch:8.1.0040: warnings from 64-bit compiler
| * | | | | | | vim-patch:8.1.0040: warnings from 64-bit compilerVVKot2021-12-20
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Warnings from 64-bit compiler. Solution: Add type casts. (Mike Williams) https://github.com/vim/vim/commit/e31e256ba1769a3a3ed7840d5cc9a01ab058b8bc
* | | | | | | Merge pull request #16786 from jamessan/cancel-previous-ciJames McCoy2021-12-25
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | ci: cancel in-progress CI if a PR is updated
| * | | | | | | ci: cancel in-progress CI if a PR is updatedJames McCoy2021-12-25
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | [skip ci]
* | | | | | | vim-patch:8.2.3873: go.mod files are not recognized (#16757)dundargoc2021-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: go.mod files are not recognized. Solution: Check for the file name. (closes vim/vim#9380) https://github.com/vim/vim/commit/82b3b4c6cf2973fe767f8e2311482af0bd95267e
* | | | | | | Merge pull request #16762 from zeertzjq/grid-truncate-pum-double-widthBjörn Linse2021-12-25
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Truncate double-width character at the end of popup menu correctly
| * | | | | | | refactor(pum_redraw): rename col -> grid_colzeertzjq2021-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is initialized to `col_off`, while in Vim this variable `col` that is used in the same places is initialized to `pum_col`. This can cause confusion in patch porting, and it caused Vim patch 8.2.1995 to be ported incorrectly. (I reverted the incorrect part in the last commit though.) Rename it to `grid_col` to make it clear that it is different from Vim's `col` variable.
| * | | | | | | fix(pum_redraw): use grid_puts_len() to truncate the textzeertzjq2021-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nvim already resizes grid to the required width, so there is no need to truncate the text in pum_redraw(). What's more, truncation is currently done incorrectly because Vim patch 8.2.1995 was ported incorrectly. This nearly reverts the truncation part of Vim patch 8.2.1995, but not the part that reduces unnecessary calls to pum_redraw(). The original PR https://github.com/vim/vim/pull/7306 didn't explain much about which part of it actually reduces redraws.
| * | | | | | | fix(screen): truncate when overwriting right half of a double-width charzeertzjq2021-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unlike the code above, this truncates the character in the same grid. This is mainly for the pum scrollbar in the next commit.
| * | | | | | | fix(screen): truncate double-width character correctlyzeertzjq2021-12-24
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | The `c = '>';` is useless here, because it is not used later. `u8c` should also need to be set to '>', and `u8cc` needs to be cleared.
* | | | | | | chore(editorconfig): set max_line_length for .h and .in files (#16775)Maverun2021-12-24
| | | | | | |
* | | | | | | fix(options): disallow empty 'fdc' and 'scl' (#16765)Sean Dewar2021-12-24
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Empty string values for these options aren't actually allowed, but check_opt_strings allows empty string options. It so happens that 'scl' handles empty string like "auto", but empty 'fdc' causes glitchiness (win_fdccol_count returns an incorrect value). Just disallow empty string values for these options completely.
* | | | | | chore: set max_line_length in editorconfig (#16755)Gregory Anders2021-12-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We established a while ago that 100 chars is our line length for both C and Lua. Not all editorconfig plugins support the "max_line_length" option, but many do (including all of the ones available for Vim/Neovim to the best of my knowledge).
* | | | | | docs: regenerate (#16742)github-actions[bot]2021-12-23
| | | | | | | | | | | | | | | | | | Co-authored-by: marvim <marvim@users.noreply.github.com>
* | | | | | Merge pull request #16754 from jamessan/vim-8.2.3874James McCoy2021-12-22
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch:8.2.3874
| * | | | | | vim-patch:8.2.3874: cannot highlight the number column for a signJames McCoy2021-12-22
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot highlight the number column for a sign. Solution: Add the "numhl" argument. (James McCoy, closes vim/vim#9381) https://github.com/vim/vim/commit/a80aad717464760a5a50ac2201ce35b24a0cf7a5
* | | | | | refactor: fix comment spacing in option.h (#16749)zeertzjq2021-12-21
| | | | | |
* | | | | | fix(api): make nil value in nvim_set_option_value clear local value (#16710)Gregory Anders2021-12-21
| | | | | | | | | | | | | | | | | | | | | | | | For special options such as 'undolevels' and 'scrolloff', this sets the local value to the special "unset" value (e.g. -12345 for 'undolevels').
* | | | | | feat(lsp): add buf_detach_client (#16250)Michael Lingelbach2021-12-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows the user to detach an active buffer from the language client. If no clients remain attached to a buffer, the on_lines callback is used to cancel nvim_buf_attach.
* | | | | | Merge pull request #16736 from dundargoc/ci/remove-unused-typo-fixJames McCoy2021-12-20
|\ \ \ \ \ \ | | | | | | | | | | | | | | ci: remove unused "squash typo" workflow
| * | | | | | ci: remove unused "squash typo" workflowDundar Göc2021-12-20
|/ / / / / /
* | | | | | Merge pull request #16734 from zeertzjq/chdir-refactorJames McCoy2021-12-20
|\ \ \ \ \ \ | | | | | | | | | | | | | | refactor: remove some chdir-related unnecessary calls and checks
| * | | | | | refactor: remove some chdir-related unnecessary calls and checkszeertzjq2021-12-20
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | xmalloc() always retuns a valid pointer. Calling os_chdir() with the same directory as the current one doesn't do anything other than wasting time.
* | | | | | fix(lsp): fix `nil`-index behavior for UTF-8 in `_str_*index_enc` methods ↵Rishikesh Vaishnav2021-12-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (#16731) Previously, the `_str_utfindex_enc` and `_str_byteindex_enc` helper functions would return `nil` when `offset_encoding == "utf-8"` and `index == nil`. Clearly, this doesn't reflect the expected behavior of the functions they're wrapping which would return the length of the line in this case. This should fix behavior with servers that use UTF-8 `offset_encoding` when applying text edits, formatting a range, and doing range code actions (though this isn't tested currently).
* | | | | | Merge pull request #16727 from dundargoc/docs/remove-good-first-issue-suggestionJames McCoy2021-12-20
|\ \ \ \ \ \ | |/ / / / / |/| | | | | docs: stop suggestion "good first issue" as a good first issue
| * | | | | docs: stop suggestion "good first issue" as a good first issueDundar Göc2021-12-19
|/ / / / / | | | | | | | | | | | | | | | | | | | | They're simply too difficult and too niche for an actual newcomer to solve them. Instead, use the "complexity:low" as a substitute.
* | | | | chore(build): add option to disable Lua bytecode compilation (#16722)Gregory Anders2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | Include a build option to explicitly disable Lua bytecode compilation. This is needed in some situations where the existing checks are not sufficient (e.g. cross compilation).
* | | | | fix(lsp): avoid attaching to unloaded buffers (#16723)Michael Lingelbach2021-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/16562 https://github.com/neovim/neovim/issues/16249 https://github.com/neovim/neovim/issues/16297 * buf_attach_client can be called on an unloaded buffer * on_attach will prematurely fail, while the language server client tracks this buffer as attached * The language server client will track this buffer as attached despite textDocument/didChange notifications not being sent to the server * Instead, check if the buffer is loaded and return early, warning via the lsp logger that buf_attach_client was called on an invalid buffer
* | | | | Merge pull request #16657 from lewis6991/unused_warnBjörn Linse2021-12-19
|\ \ \ \ \ | |/ / / / |/| | | | chore(): suppress -Wunused-but-set-variable warns
| * | | | chore: suppress -Wunused-but-set-variable warnsLewis Russell2021-12-19
| | | | |
* | | | | fix(build): check that LuaJIT has required modules for compilation (#16714)Gregory Anders2021-12-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apparently some installations of LuaJIT do not have the "jit.bcsave" module which is required for pre-compilation. Check that this module exists before using LuaJIT as luac and if it doesn't, skip compiling bytecode.
* | | | | fix: update last cursor on first CursorMoved (#16698)Michael Lingelbach2021-12-18
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/16625 https://github.com/neovim/neovim/issues/12923 The first defined CursorMoved autocommand will immediately fire if the cursor has previously moved upon definition of the autocommand. Plugins add dummy autocommands such as: ```lua autocmd CursorMoved * execute '' ``` to avoid this behavior. Instead, when defining a new CursorHold autocommand, force update the last cursor position. See https://github.com/vim/vim/issues/2053
* | | | build(deps): bump luajit to commit 1d20f33 (#16712)Christian Clason2021-12-18
| | | | | | | | | | | | fixes a bug with jit.bcsave on Windows (cf. #16631)