Commit message (Collapse) | Author | Age | ||
---|---|---|---|---|
... | ||||
* | | | vim-patch:9.0.1246: code is indented more than necessary (#22006) | zeertzjq | 2023-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> | |||
* | | | vim-patch:9.0.1242: code for :runtime completion is not consistent | zeertzjq | 2023-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 improved | zeertzjq | 2023-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} argument | zeertzjq | 2023-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 :runtime | zeertzjq | 2023-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) | zeertzjq | 2023-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) | zeertzjq | 2023-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) | zeertzjq | 2023-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" ↵ | zeertzjq | 2023-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 ↵ | zeertzjq | 2023-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> | |||
* | | Merge pull request #21885 from lewis6991/refactor/options | Lewis Russell | 2023-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 formatlistpat | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_expropt | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): remove some redundant parens | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_option_listflags | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): add did_set_string_option_for | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 52 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 51 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): remove some simple did_set_* functions 2 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): remove some simple did_set_* functions | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 50 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 49 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 48 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 47 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 46 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 45 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 44 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 43 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 42 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 41 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 40 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 39 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 38 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 37 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 36 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 35 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 34 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 33 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 32 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 31 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 30 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 29 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 28 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 27 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 26 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 25 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 24 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 23 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 22 | Lewis Russell | 2023-01-25 | |
| | | | ||||
| * | | refactor(optionstr.c): break up did_set_string_option 21 | Lewis Russell | 2023-01-25 | |
| | | |