| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
Problem: Misleading variable name for error message.
Solution: Change "name" to "number". (closes #12345)
|
| |
|
|
|
|
| |
Problem: Code indenting is confused by macros.
Solution: Put semicolon after the macros instead of inside. (Ozaki Kiichi,
closes #12257)
|
| |
|
|
|
| |
Problem: Unused variables and functions.
Solution: Delete items and adjust #ifdefs. (Dominique Pellé, closes #12145)
|
| |
|
|
|
|
| |
Problem: find_file_in_path() is not reentrant.
Solution: Instead of global variables pass pointers to the functions.
(closes #12093)
|
| |
|
|
|
| |
Problem: Functions without arguments are not always declared properly.
Solution: Use "(void)" instead of "()". (Yegappan Lakshmanan, closes #12031)
|
| |
|
|
|
| |
Problem: Handling new value of an option has a long "else if" chain.
Solution: Use a function pointer. (Yegappan Lakshmanan, closes #12015)
|
| |
|
|
|
|
| |
Problem: Using isalpha() adds dependency on current locale.
Solution: Do not use isalpha() for recognizing a URL or the end of an Ex
command. (closes #11835)
|
| |
|
|
|
| |
Problem: :interface is not implemented yet.
Solution: Implement the basics of :interface.
|
| |
|
|
|
|
| |
Problem: Code is indented more than needed.
Solution: Use an early return to reduce indentation. (Yegappan Lakshmanan,
closes #11769)
|
| |
|
|
|
|
| |
Problem: Code is indented more than needed.
Solution: Use an early return to reduce indenting. (Yegappan Lakshmanan,
closes #11758)
|
| |
|
|
|
|
|
| |
Problem: Classes are not documented or implemented yet.
Solution: Make the first steps at documenting Vim9 objects, classes and
interfaces. Make initial choices for the syntax. Add a skeleton
implementation. Add "public" and "this" in the command table.
|
| |
|
|
|
| |
Problem: Using one window for executing autocommands is insufficient.
Solution: Use up to five windows for executing autocommands.
|
| |
|
|
|
| |
Problem: Error message for layout change does not match action.
Solution: Pass the command to where the error is given. (closes #11573)
|
| |
|
|
|
| |
Problem: Restoring window after WinScrolled may fail.
Solution: Lock the window layout when triggering WinScrolled.
|
| |
|
|
|
|
| |
Problem: Solution for "!!sort" in closed fold is not optimal.
Solution: Use a different range instead of the subtle difference in handling
a range with an offset. (issue #11487)
|
| |
|
|
|
| |
Problem: Compiler warning for unused variable.
Solution: Add #ifdef.
|
| |
|
|
|
| |
Problem: "!!sort" in a closed fold sorts too many lines.
Solution: Round to end of fold after adding the line count. (closes #11487)
|
| |
|
|
|
|
| |
Problem: :confirm does not work properly for a terminal buffer.
Solution: Handle :confirm for a terminal buffer differently. (Yee Cheng
Chin, closes #11312)
|
| |
|
|
|
| |
Problem: Debugger does not display the whole command.
Solution: Set ea.cmd before checking for a breakpoint.
|
| |
|
|
|
|
| |
Problem: "export def" does not work in a nested block.
Solution: Do not handle "export" with a separate function but in the same
command stack. (closes #11304)
|
| |
|
|
|
|
| |
Problem: Too many #ifdefs.
Solution: Graduate the +cmdwin feature. Now the tiny and small builds are
equal, drop the small build. (Martin Tournoij, closes #11268)
|
| |
|
|
|
| |
Problem: Syntax of commands in Vim9 script depends on +eval feature.
Solution: Use same syntax with and without the +eval feature.
|
| |
|
|
|
|
|
| |
Problem: Evaluating "expr" options has more overhead than needed.
Solution: Use call_simple_func() for 'foldtext', 'includeexpr', 'printexpr',
"expr" of 'spellsuggest', 'diffexpr', 'patchexpr', 'balloonexpr',
'formatexpr', 'indentexpr' and 'charconvert'.
|
| |
|
|
|
|
| |
Problem: Crash when closing a tabpage and buffer is NULL.
Solution: Adjust how autocommands are triggered when closing a window.
(closes #11198, closes #11197)
|
| |
|
|
|
| |
Problem: When at the command line :redrawstatus does not work well.
Solution: Only update the statuslines instead of the screen. (closes #11180)
|
| |
|
|
|
|
| |
Problem: Cannot redraw the status lines when editing a command.
Solution: Only postpone the redraw when messages have scrolled.
(closes #11170)
|
| |
|
|
|
|
| |
Problem: Command line cleared when using :redrawstatus in CmdlineChanged
autocommand event.
Solution: Postpone the redraw. (closes #11162)
|
| |
|
|
|
| |
Problem: fullcommand() only works for the current script version.
Solution: Add an optional argument for the script version.
|
| |
|
|
|
| |
Problem: Trying to declare g:variable gives confusing error.
Solution: Give a better error message. (closes #11108)
|
| |
|
|
|
|
|
| |
Problem: "for" and "while" not recognized after :vim9cmd and :legacy.
(Emanuele Torre)
Solution: Recognize all the command modifiers. (closes #11087)
Add a test to check the list of modifiers.
|
| |
|
|
|
| |
Problem: Crash when invalid line number on :for is ignored.
Solution: Do not check breakpoint for non-existing line.
|
| |
|
|
|
| |
Problem: :echowindow does not work in a compiled function.
Solution: Handle the expression at compile time.
|
| |
|
|
|
|
| |
Problem: :horizontal modifier not fully supported.
Solution: Also use :horizontal for completion and user commands.
(closes #11025)
|
| |
|
|
|
|
| |
Problem: ":wincmd =" equalizes in two directions.
Solution: Make ":vertical wincmd =" equalize vertically only and
":horizontal wincmd =" equalize horizontally only.
|
| |
|
|
|
| |
Problem: The 'cmdheight' zero support causes too much trouble.
Solution: Revert support for 'cmdheight' being zero.
|
| |
|
|
|
| |
Problem: Clearing screen causes flicker.
Solution: Do not clear but redraw in more cases. Add () to "wait_return".
|
| |
|
|
|
| |
Problem: :echomsg doesn't work properly with cmdheight=0.
Solution: Improve scrolling and displaying.
|
| |
|
|
|
| |
Problem: No good reason why the "gf" command is not in the tiny version.
Solution: Graduate the file_in_path feature.
|
| |
|
|
|
| |
Problem: Too many #ifdefs.
Solution: Make some functions always available.
|
| |
|
|
|
|
| |
Problem: Mechanism to prevent recursive screen updating is incomplete.
Solution: Add "redraw_not_allowed" and set it in build_stl_str_hl().
(issue #10952)
|
| |
|
|
|
| |
Problem: Redraw flags are not named specifically.
Solution: Prefix "UPD_" to the flags, for UPDate_screen().
|
| |
|
|
|
| |
Problem: Cannot build with small features.
Solution: Check for E1170 only with FEAT_EVAL.
|
| |
|
|
|
| |
Problem: Giving E1170 only in an expression is confusing.
Solution: Give E1170 for any "#{ comment". (closes #10855)
|
| |
|
|
|
| |
Problem: Typos in comments, wrapping lines.
Solution: Adjust comments. Wrap lines.
|
| |
|
|
|
|
| |
Problem: Accessing beyond allocated memory when using the cmdline window in
Ex mode.
Solution: Use "*" instead of "'<,'>" for Visual mode.
|
| |
|
|
|
| |
Problem: Read past the end of the first line with ":0;'{".
Solution: When on line zero check the column is valid for line one.
|
| |
|
|
|
| |
Problem: Substitute may overrun destination buffer.
Solution: Disallow switching buffers in a substitute expression.
|
| |
|
|
|
|
| |
Problem: Error message for unknown command may mention the command twice.
(Malcolm Rowe)
Solution: Add the did_append_cmd flag. (closes #10570)
|
| |
|
|
|
| |
Problem: Using "'<,'>" in Ex mode may compare unrelated pointers.
Solution: Set eap->cmd to "+" only later.
|
| |
|
|
|
| |
Problem: Value of cmod_verbose is a bit complicated to use.
Solution: Use zero for not set, value + 1 when set. (closes #10564)
|
