aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | fix(winhl): do not crash when unsetting winhl in just opened windowbfredl2022-08-18
| |/ / | | | | | | | | | fixes #19823
* | | vim-patch:8.1.2082: rename popupmnu.* to popupmenu.* (#19829)zeertzjq2022-08-18
| | | | | | | | | | | | | | | | | | | | | vim-patch:8.1.2082: some files have a weird name to fit in 8.3 characters Problem: Some files have a weird name to fit in 8.3 characters. Solution: Use a nicer names. https://github.com/vim/vim/commit/30e8e73506e4522ef4aebf7d525c0e6ffe8805fd
* | | docs(lsp): remove lsp.buf_request from docs (#19738)Mathias Fußenegger2022-08-18
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This starts a soft phase-out of `buf_request`. `buf_request` is quite error prone: - Positional `params` depend on the client because of the `offset_encoding`. Currently if there is one client using UTF-8 offset encoding and another using UTF-16, the positions in the request are wrong for one of the clients. To solve this the params would need to be created per client instead of once for all of them. - `handler` is called *per* client but many users of it assume it is only called once. This can lead to a "select n + 1" kind of problem, where the handler makes another call to `buf_request`, multiplying the amount of requests. (There are in fact still some places where this happens in core) Or it leads to erratic behavior if called multiple times (E.g. the quicklist list flickering & being overwritten) (See hover or references implementation) `buf_request_all` returns an aggregate of the responses which is more sensible as it avoids this problem. For off-spec extensions it also has the problem that it sends requests to clients which cannot handle a given request. Given that `buf_request` is in use by a lot of plugins this starts a soft-phase out. Planned Steps: - Remove from docs - Provide an alternative, either `buf_request_all`, maybe with extensions (params being a function), or an entirely new method. - Mark as deprecated in 0.9 - Remove in 0.10 To note: - `buf_request_all` currently isn't ideal either because it suffers from the `params` problem as well. - This implies that the `vim.lsp.with` pattern will die, because the global handlers as they are don't fit a multi-client model, as most of the time an aggregate is needed.
* | Merge pull request #19824 from bfredl/msgsep_allthethingsbfredl2022-08-18
|\ \ | | | | | | fix(tests): remove irrelevant usage of display-=msgsep
| * | fix(tests): remove irrelevant usage of display-=msgsepbfredl2022-08-17
|/ / | | | | | | | | | | | | | | These were just added to avoid churn when changing the default of 'display'. To simplify message handling logic, we might want to remove support for printing messages in default_grid later on. This would allow things like printing error messages safely in the middle of redraw, or a future graduation of the 'multigrid' feature.
* | Merge pull request #13457 from bfredl/färgtemabfredl2022-08-17
|\ \ | | | | | | feat(highlight): support highlight namespaces per window
| * | feat(ui): allow to set the highlight namespace per windowbfredl2022-08-17
|/ / | | | | | | | | - reimplement 'winhl' in terms of highlight namespaces - check for EOF in screen tests (to indicate a likely crash)
* | refactor: remove some unused includes (#19820)zeertzjq2022-08-17
| | | | | | Replace grid.h in screen.h and screen.h in buffer.h with grid_defs.h
* | feat: allow :wincmd to accept a count (#19815)Famiu Haque2022-08-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let :wincmd command accept a count like what its documentation suggests. Previously it could only accept a range, which led to some ambiguity on which attribute should be used when executing :wincmd using nvim_cmd. Closes #19662. Also fix a typo in a related Vim test: vim-patch:9.0.0223: typo in diffmode test Problem: Typo in diffmode test. Solution: Fix the typo. (closes vim/vim#10932) https://github.com/vim/vim/commit/5fd6ab820b4a0aaa5c6020852f39d118375fab49
* | docs(lsp): rename on-list-handler to lsp-on-list-handler (#19813)Jonas Strittmatter2022-08-17
| | | | | | | | This makes it easier to find documentation about the on-list-handler when starting the search term with "lsp".
* | vim-patch:8.2.1262: src/ex_cmds.c file is too big (#19811)zeertzjq2022-08-17
| | | | | | | | | | | | Problem: src/ex_cmds.c file is too big. Solution: Move help related code to src/help.c. (Yegappan Lakshmanan, closes vim/vim#6506) https://github.com/vim/vim/commit/f868ba89039045b25efe83d12ca501d657e170e8
* | vim-patch:8.2.3699: the +title feature adds a lot of #ifdef but little code ↵zeertzjq2022-08-17
| | | | | | | | | | | | | | (#19809) Problem: The +title feature adds a lot of #ifdef but little code. Solution: Graduate the +title feature. https://github.com/vim/vim/commit/651fca85c71a4c5807f8f828f9ded30fbd754325
* | Merge pull request #19801 from Shougo/vim-9.0.0190zeertzjq2022-08-17
|\ \ | | | | | | vim-patch:9.0.{0190,0191,0192}: cmdheight=0 fixes
| * | vim-patch:9.0.0191: messages test fails; window size incorrectShougo Matsushita2022-08-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Messages test fails; window size incorrect when 'cmdheight' is made smaller. Solution: Properly cleanup after test with cmdheight zero. Resize windows correctly when 'cmdheight' gets smaller. https://github.com/vim/vim/commit/d4cf9fc53e0b1d36e84d28ecd5595a6f102f325e N/A patches for version.c: vim-patch:9.0.0192: possible invalid memory access when 'cmdheight' is zero Problem: Possible invalid memory access when 'cmdheight' is zero. (Martin Tournoij) Solution: Avoid going over the end of w_lines[] when w_height is Rows. (closes vim/vim#10882) https://github.com/vim/vim/commit/fdc5d17d58cc9c9edc9fb2816e1afaabc531bf1e
| * | vim-patch:9.0.0190: the way 'cmdheight' can be made zero is inconsistentShougo Matsushita2022-08-17
|/ / | | | | | | | | | | | | Problem: The way 'cmdheight' can be made zero is inconsistent. Solution: Only make 'cmdheight' zero when setting it explicitly, not when resizing windows. (closes vim/vim#10890) https://github.com/vim/vim/commit/f797e309caff48f7a56c73b16e62ff67c4dcbdd6
* | Merge pull request #19808 from zeertzjq/vim-9.0.0220zeertzjq2022-08-17
|\ \ | | | | | | vim-patch:9.0.{0220,0222}
| * | vim-patch:9.0.0222: no good reason why text objects are only in larger buildszeertzjq2022-08-17
| | | | | | | | | | | | | | | | | | Problem: No good reason why text objects are only in larger builds. Solution: Graduate +textobjects. https://github.com/vim/vim/commit/887748742deae3d6de7aa0fdbb042afe1ccf5e7a
| * | vim-patch:9.0.0220: invalid memory access with for loop over NULL stringzeertzjq2022-08-17
|/ / | | | | | | | | | | Problem: Invalid memory access with for loop over NULL string. Solution: Make sure mb_ptr2len() consistently returns zero for NUL. https://github.com/vim/vim/commit/f6d39c31d2177549a986d170e192d8351bd571e2
* | refactor: change pre-decrement/increment to post (#19799)Lewis Russell2022-08-16
| | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* | refactor(signs): handle non-sign attrs separately (#19784)Lewis Russell2022-08-16
| |
* | docs: update .gitattributes (#19800)Lewis Russell2022-08-16
| |
* | fix(api): nvim_exec and nvim_cmd restore msg_col when capturing output (#19789)zeertzjq2022-08-16
| | | | | | | | This matches the code in execute_common(), preventing messages after the API call from being printed at the wrong column.
* | refactor(eval.c): resolve all clint issues (#19774)Lewis Russell2022-08-16
| |
* | vim-patch:e1f3fd1d02e3 (#19796)Christian Clason2022-08-16
| | | | | | | | Update runtime files https://github.com/vim/vim/commit/e1f3fd1d02e3f5fe6d2b6d82687c6846b8e500f8
* | vim-patch:8.2.3888: the argument list may contain duplicates (#19795)zeertzjq2022-08-16
| | | | | | | | | | | | | | Problem: The argument list may contain duplicates. Solution: Add the :argdedeupe command. (Nir Lichtman, closes vim/vim#6235) https://github.com/vim/vim/commit/73a024209cbfbd5b39a2e974084d807c6131e2ed Use latest index.txt :argdedupe doc from Vim.
* | test: use poke_eventloop() instead of sleep(10) where possible (#19794)zeertzjq2022-08-16
| | | | | | | | Using sleep(10) to wait for typeahead to finish is flaky, especially on macOS, where legacy/global_spec.lua has failed several times.
* | vim-patch:8.1.2042: the evalfunc.c file is too big (#19792)zeertzjq2022-08-16
| | | | | | | | | | | | | | Problem: The evalfunc.c file is too big. Solution: Move getchar() and parse_queued_messages() to getchar.c. https://github.com/vim/vim/commit/9c658c9eacbd97e2c071f652a0155f71db94c0f3 Omit parse_queued_messages(): Nvim does not have MESSAGE_QUEUE.
* | vim-patch:8.1.1869: code for the argument list is spread out (#19791)zeertzjq2022-08-16
| | | | | | | | | | | | Problem: Code for the argument list is spread out. Solution: Put argument list code in arglist.c. (Yegappan Lakshmanan, closes vim/vim#4819) https://github.com/vim/vim/commit/4ad62155a1015751a6645aaecd94b02c94c8934b
* | vim-patch:9.0.0216: undo earlier test sometimes fails on MS-Windows (#19790)zeertzjq2022-08-16
| | | | | | | | | | Problem: Undo earlier test sometimes fails on MS-Windows. Solution: Use another file name. https://github.com/vim/vim/commit/cce293f87beb57a75ff738fade7fafadbc4a78a9
* | Merge pull request #19781 from zeertzjq/source-lua-estackzeertzjq2022-08-16
|\ \ | | | | | | fix(source): fix expand('<sfile>') no longer works for Lua
| * | fix(source): fix expand('<sfile>') no longer works for Luazeertzjq2022-08-16
| | |
| * | vim-patch:9.0.0217: 'shellslash' works differently when sourcing a script againzeertzjq2022-08-16
|/ / | | | | | | | | | | Problem: 'shellslash' works differently when sourcing a script again. Solution: Use the name from the script item. (closes vim/vim#10920) https://github.com/vim/vim/commit/5a4fff4d948cd12a5cf5f637ad2c561815a77d8e
* | Merge pull request #19635 from dundargoc/build/windowsChristian Clason2022-08-15
|\ \ | | | | | | build: windows-specific cmake fixes
| * | build: only use CMAKE_BUILD_TYPE for single-config generatorsdundargoc2022-08-13
| | | | | | | | | | | | | | | CMAKE_BUILD_TYPE is ignored for multi-config generators and creates a warning that it's unused.
| * | build(MSVC): don't add non-MSVC compiler optionsdundargoc2022-08-13
| | | | | | | | | | | | | | | This will prevent warnings of the type "ignoring unknown option '-fPIC'" when using MSVC.
| * | build(windows): fix target check-single-includesdundargoc2022-08-13
| | | | | | | | | | | | | | | Add missing includes and exclude os-specific files depending on platform when using the target.
| * | build: bump minimum cmake version for all dependencies to 3.10dundargoc2022-08-13
| | | | | | | | | | | | | | | This removes cmake policy warning for CMP0053 on windows and ensures the build works correctly for newer cmake policies.
* | | Merge pull request #19785 from zeertzjq/vim-8.2.1803zeertzjq2022-08-15
|\ \ \ | | | | | | | | vim-patch:8.2.{1803,3345,3392}
| * | | vim-patch:8.2.3392: augroup completion escapes regexp pattern characterszeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | Problem: augroup completion escapes regexp pattern characters. Solution: Do not escape the augroup name. (closes vim/vim#8826) https://github.com/vim/vim/commit/b4d82e2a8d610c00139a74970df772eece2daf1c
| * | | vim-patch:8.2.3345: some code not covered by testszeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Some code not covered by tests. Solution: Add a few more tests. (Dominique Pellé, closes vim/vim#8757) https://github.com/vim/vim/commit/bfb2bb16bc69441fa3ec13caacb2c94637a8a0ec
| * | | vim-patch:8.2.1803: a few failures are not testedzeertzjq2022-08-15
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: A few failures are not tested. Solution: Test a few failures. (Dominique Pellé, closes vim/vim#7075) https://github.com/vim/vim/commit/afe8cf617013fd8c3f0189f1e1fa7a2a6a8f7511 Cherry-pick a line in Test_argdelete() from patch 8.2.1736.
* | | Merge pull request #19440 from zeertzjq/vim-8.2.1786zeertzjq2022-08-15
|\ \ \ | | | | | | | | vim-patch:8.2.{1786,1799}: Normal mode commands not fully tested
| * | | fix(redraw): make sure :redraw! redraws command linezeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | In Vim :redraw! uses CLEAR, which calls screenclear(), which sets redraw_cmdline. In Nvim :redraw! uses NOT_VALID, which does not set redraw_cmdline automatically, so set it explicitly.
| * | | vim-patch:8.2.1799: some Normal mode commands not fully testedzeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some Normal mode commands not fully tested. Solution: Add a few more tests. (Yegappan Lakshmanan, closes vim/vim#7073) https://github.com/vim/vim/commit/d1ad99b65470d3e754f6a0588a6b0dc2214a1eab Cherry-pick test_registers.vim change from patch 8.2.0644.
| * | | vim-patch:8.2.1786: various Normal mode commands not fully testedzeertzjq2022-08-15
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: Various Normal mode commands not fully tested. Solution: Add more tests. (Yegappan Lakshmanan, closes vim/vim#7059) https://github.com/vim/vim/commit/8a9bc95eaec53f4e0c951ff8f2686ae5113a5709 Cherry-pick Test_normal_gdollar_cmd() change from patch 8.2.0540.
* | | Merge pull request #19098 from zeertzjq/vim-8.1.0822zeertzjq2022-08-15
|\ \ \ | |_|/ |/| | vim-patch:8.1.{partial:0822,1189,1192},8.2.5109: 'showmode' fixes
| * | vim-patch:8.2.5109: mode not updated after CTRL-O CTRL-C in Insert modezeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | Problem: Mode not updated after CTRL-O CTRL-C in Insert mode. Solution: Set redraw_mode and use it. (closes vim/vim#10581) https://github.com/vim/vim/commit/7a1d32809bb5c1527314000983e75125d79192e0
| * | vim-patch:8.1.1192: mode is not cleared when leaving Insert mode with mapped Esczeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | Problem: Mode is not cleared when leaving Insert mode with mapped Esc. Solution: Clear the mode when redraw_cmdline is set. (closes vim/vim#4269) https://github.com/vim/vim/commit/4c25bd785aa8b565bf973cbba12ed36b76daaa4f
| * | vim-patch:8.1.1189: mode is not cleared when leaving Insert modezeertzjq2022-08-15
| | | | | | | | | | | | | | | | | | Problem: Mode is not cleared when leaving Insert mode. Solution: Clear the mode when got_int is set. (Ozaki Kiichi, closes vim/vim#4270) https://github.com/vim/vim/commit/abc7c7fc5a098374f5543a237e6c9dd918848b34
| * | fix(getchar): flush screen before doing a blocking waitzeertzjq2022-08-15
| | |