aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | refactor(fileio.c): reduce scope of localsLewis Russell2023-01-26
| | |
* | | vim-patch:9.0.1246: code is indented more than necessary (#22006)zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code is indented more than necessary. Solution: Use an early return where it makes sense. (Yegappan Lakshmanan, closes vim/vim#11887) https://github.com/vim/vim/commit/142ed77898facf8f423fee2717efee1749c55f9a Omit function_using_block_scopes(): only affects Vim9 script. Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
* | | docs: clarify :runtime behavior without [where] again (#22003)zeertzjq2023-01-26
| | | | | | | | | | | | The behavior changed again after #15867, change the docs to describe latest behavior.
* | | test: exepath() returns correct path with cmd.exe, powershell #21928Anton Kriese2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test(exepath): test if exepath returns correct path with multiple Windows shells This test covers the changes from #21175 where exepath() is set to prefer file extensions in powershell.exe aswell as in cmd.exe. In both shells, the file with a valid extension should be returned instead of the extensionless file.
* | | Merge pull request #22002 from zeertzjq/vim-9.0.1227zeertzjq2023-01-26
|\ \ \ | | | | | | | | vim-patch:9.0.{1227,1231,1238,1242}: cmdline completion for :runtime
| * | | test: add test for :runtime completion for .luazeertzjq2023-01-26
| | | |
| * | | vim-patch:9.0.1242: code for :runtime completion is not consistentzeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code for :runtime completion is not consistent. Solution: Make code for cmdline expansion more consistent. (closes vim/vim#11875) https://github.com/vim/vim/commit/b0d45ec67f4976318f199a7929ad3bcf93686fd0
| * | | vim-patch:9.0.1238: :runtime completion can be further improvedzeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: :runtime completion can be further improved. Solution: Also complete the {where} argument values and adjust the completion for that. (closes vim/vim#11874) https://github.com/vim/vim/commit/5c8771bc5a2be123ab8e6325fa60ed524e8efb09
| * | | vim-patch:9.0.1231: completion of :runtime does not handle {where} argumentzeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Completion of :runtime does not handle {where} argument. Solution: Parse the {where} argument. (closes vim/vim#11863) https://github.com/vim/vim/commit/3770f4c9cde7b5fcd10b6fa2e665cd0b69450fb2
| * | | vim-patch:9.0.1227: no cmdline completion for :runtimezeertzjq2023-01-26
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: No cmdline completion for :runtime. Solution: Add completion for :runtime. (closes vim/vim#11853, closes vim/vim#11447) Improve the resulting matches. https://github.com/vim/vim/commit/a6759381a590b2d395e05b109ca9ccfc356be5a8
* | | refactor(intro): avoid Coverity warning (#22000)zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | refactor(intro): avoid coverity warning Problem: Coverity warns about overwriting "mesg" leaking memory. Solution: Make it clear that "mesg" will not be overwritten.
* | | refactor(options): don't pass negative number to illegal_char() (#21999)zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | This only changes the error messages for an unexpected Unicode char in an option to show its first byte instead of some special keycode. The second argument of vim_strchr() usually doesn't matter, but it may be better to consistently cast to uint8_t.
* | | vim-patch:9.0.1245: code is indented more than necessary (#21998)zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code is indented more than necessary. Solution: Use an early return where it makes sense. (Yegappan Lakshmanan, closes vim/vim#11879) https://github.com/vim/vim/commit/032713f8299abd92fcfb1e490d1ae5c1ecadde41 Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
* | | vim-patch:9.0.1243: :setglobal cannot use script-local function for "expr" ↵zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | option (#21997) Problem: :setglobal cannot use script-local function for "expr" option. Solution: Use the pointer to the option value properly. (closes vim/vim#11883) https://github.com/vim/vim/commit/01d4efe2e87632aa085897d3d64e27585908f977
* | | vim-patch:9.0.1244: cursor displayed in wrong position when leaving Insert ↵zeertzjq2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mode (#21996) Problem: Cursor briefly displayed in a wrong position when pressing Esc in Insert mode after autoindent was used. Solution: Do not adjust the cursor position for assumed deleted white space if text is following. (closes vim/vim#11877) https://github.com/vim/vim/commit/0f843ef091eceb470caece1d90fdfe08926fe076 Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | | build: use cmake to build libvterm on all platform (#21986)dundargoc2023-01-26
| | | | | | | | | | | | | | | | | | | | | | | | Also remove Libvterm-tbl2inc_c.cmake as it's not required. It's used to generate files that are already provided by the Libvterm project by default. It's also not really something we need to concern ourselves with as it's more of an authoring tool for the Libvterm creator as mentioned in https://github.com/neovim/neovim/pull/21986#issuecomment-1403733054.
* | | ci: fix lintc to use external dependencies instead of bundled (#21995)dundargoc2023-01-25
| | | | | | | | | | | | | | | | | | | | | Use the bundled libvterm dependency as the external package is outdated, with the hopes of being able to use the external package once its version meets our required version. Co-authored-by: Christian Clason <c.clason@uni-graz.at>
* | | fix(man.lua): open in current window if it's already a man page (#21987)0xAdk2023-01-25
|/ / | | | | This matters when there are multiple man page windows open.
* | Merge pull request #21885 from lewis6991/refactor/optionsLewis Russell2023-01-25
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problems: - Scope of local variables in options code is too large. - did_set_string_option() is too large (>1000LOC). - Setting options for a particular window or buffer requires a changing context (assigning curwin/curbuf). Solutions: - Reduce the scope of local variables. - Break up did_set_string_option so it doesn't contain specific logic about each individual option (1038 LOC -> 310 LOC). - Begin work on making functions not depend on curbuf or curwin and pass window or buffer handles explicitly.
| * | refactor(optionstr.c): move handling of formatlistpatLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_exproptLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): remove some redundant parensLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_option_listflagsLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): add did_set_string_option_forLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 52Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 51Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): remove some simple did_set_* functions 2Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): remove some simple did_set_* functionsLewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 50Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 49Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 48Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 47Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 46Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 45Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 44Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 43Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 42Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 41Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 40Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 39Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 38Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 37Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 36Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 35Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 34Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 33Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 32Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 31Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 30Lewis Russell2023-01-25
| | |
| * | refactor(optionstr.c): break up did_set_string_option 29Lewis Russell2023-01-25
| | |