Commit message (Collapse) | Author | Age | ||
---|---|---|---|---|
... | ||||
| * | | refactor(option.c): factor out option prefix parsing | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): factor out set op parsing | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): use skiptowhite_esc | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): change nextchar to uint8_t | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): remove goto | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): int -> bool | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): reduce scope or startarg | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): reduce scope or errbuf | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): reduce scope or errmsg | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): factor out loop code from do_set() | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): factor out common skip check | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): simplify do_set_string | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): add do_set_bool | Lewis Russell | 2023-01-26 | |
| | | | ||||
| * | | refactor(option.c): add do_set_num | Lewis Russell | 2023-01-26 | |
| | | | ||||
* | | | build: find unibilium without relying on libfindmacros (#22015) | dundargoc | 2023-01-27 | |
| | | | | | | | | | | | | This will remove the warning about the find module not providing a version. | |||
* | | | fix(test): unset XDG_CONFIG_HOME when running oldtest | Lewis Russell | 2023-01-27 | |
| | | | | | | | | | | | | - also fix test_taglist.vim for users running with a tags file created in runtime/doc | |||
* | | | build: check if libvterm version meets requirement (#22010) | dundargoc | 2023-01-26 | |
| | | | | | | | | | | | | The vterm.h file only specifies major and minor version, but not patch, meaning that requiring a specific patch number isn't currently possible. | |||
* | | | refactor(fileio.c): reduce scope of locals | Lewis Russell | 2023-01-26 | |
| | | | ||||
* | | | 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 | |
| | | |