| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
Problem: Some code blocks are nested too deep.
Solution: Bail out earlier. (Yegappan Lakshmanan, closes #11058)
|
| |
|
|
|
|
| |
Problem: GUI: when CTRL-D is mapped in Insert mode it gets inserted.
(Yasuhiro Matsumoto)
Solution: Also recognize modifier starting with CSI. (closes #11057)
|
| |
|
|
|
| |
Problem: Covertity still complains about using return value of getc().
Solution: Check for EOF.
|
| |
|
|
|
| |
Problem: Coverity complains about unused value.
Solution: Use the value.
|
| |
|
|
|
| |
Problem: Freeing the wrong string on failure.
Solution: Adjust the argument. Reorder the code.
|
| |
|
|
|
|
| |
Problem: Writefile test leaves files behind.
Solution: Fix the file names of files to be deleted. (Dominique Pellé,
closes #11056)
|
| |
|
|
|
| |
Problem: Deleting files in tests is a hassle.
Solution: Use the new 'D' flag of writefile().
|
| | |
|
| |
|
|
|
| |
Problem: Deleting files in tests is a hassle.
Solution: Use the new 'D' flag of writefile().
|
| | |
|
| |
|
|
|
|
| |
Problem: Cleaning up after writefile() is a hassle.
Solution: Add the 'D' flag to defer deleting the written file. Very useful
in tests.
|
| |
|
|
|
| |
Problem: Compiler warning for uninitialized variable. (Tony Mechelynck)
Solution: Initialize it.
|
| |
|
|
|
| |
Problem: Argument assignment does not work.
Solution: Skip over "=".
|
| |
|
|
|
| |
Problem: Clang warns for dead assignments.
Solution: Adjust the code. (Yegappan Lakshmanan, closes #11048)
|
| |
|
|
|
| |
Problem: The footer feature is unused.
Solution: Remove FEAT_FOOTER and code.
|
| |
|
|
|
| |
Problem: Coverity still complains about dropping sign of character.
Solution: Add intermediate variable.
|
| |
|
|
|
|
| |
Problem: Coverity warns for NULL check and unused return value.
Solution: Remove the NULL check, it was already checked earlier. Add (void)
to ignore the return value.
|
| |
|
|
|
| |
Problem: MS-Windows: "%T" time format does not appear to work.
Solution: Use "%H:%M:%S" instead.
|
| |
|
|
|
| |
Problem: Compiler warning for uninitialized variable.
Solution: Initialize the variable. (John Marriott)
|
| |
|
|
|
| |
Problem: Cleaning up afterwards can make a function messy.
Solution: Add the :defer command.
|
| |
|
|
|
| |
Problem: A failing flaky test doesn't mention the time.
Solution: Add the time for debugging. Improve error message.
|
| |
|
|
|
| |
Problem: Old Coverity warning for using NULL pointer.
Solution: Bail out if dictionary allocation fails.
|
| |
|
|
|
| |
Problem: Coverity complains about dropping sign of character.
Solution: Add explicit type cast.
|
| |
|
|
|
| |
Problem: Cannot use import->Func() in lambda. (Israel Chauca Fuentes)
Solution: Adjust how an expression in a lambda is parsed. (closes #11042)
|
| |
|
|
|
| |
Problem: File name used in test is unusual.
Solution: Rename it. (Dominique Pellé, closes #11044)
|
| |
|
|
|
| |
Problem: Clang static analyzer gives warnings.
Solution: Avoid the warnings. (Yegappan Lakshmanan, closes #11043)
|
| |
|
|
|
| |
Problem: Common names in test files causes tests to be flaky.
Solution: Use more specific names.
|
| |
|
|
|
| |
Problem: Expanding ":e %" does not work for remote files.
Solution: If the "%" or "#" file does not exist add the expansion anyway.
|
| |
|
|
|
|
|
| |
Problem: Removing a listener may result in a memory leak and remove
subsequent listerns.
Solution: Init the "prev" pointer only once. (Yegappan Lakshmanan,
closes #11039)
|
| |
|
|
|
| |
Problem: Crash when invalid line number on :for is ignored.
Solution: Do not check breakpoint for non-existing line.
|
| |
|
|
|
|
| |
Problem: Error message for wrong argument type is not specific.
Solution: Include more information in the error. (Yegappan Lakshmanan,
closes #11037)
|
| |
|
|
|
|
| |
Problem: 'breakindent' does not indent non-lists with
"breakindentopt=list:-1".
Solution: Adjust indent computation. (Maxim Kim, closes #11038)
|
| |
|
|
|
|
|
| |
Problem: 'linebreak' interferes with text property highlight if there is
syntax highlighting.
Solution: Check the text prop attributes after combining with syntax
attributes. (closes #11035)
|
| |
|
|
|
|
| |
Problem: :echowindow sets the in_echowindow flag too early.
Solution: Set in_echowindow only when outputting the text. (Yasuhiro
Matsumoto, closes #11033)
|
| |
|
|
|
|
| |
Problem: Check for uppercase char in autoload name is wrong, it checks the
name of the script.
Solution: Remove the check. (closes #11031)
|
| |
|
|
|
| |
Problem: MS-Windows: starting a python server for test sometimes fails.
Solution: Increase the waiting time for the port.
|
| |
|
|
|
| |
Problem: Missing entry in switch.
Solution: Add ISN_ECHOWINDOW.
|
| |
|
|
|
| |
Problem: using :echowindow in a timer clears part of message
Solution: Do not use msg_clr_eos().
|
| |
|
|
|
| |
Problem: Message window may obscure the command line.
Solution: Reduce the maximum height of the message window.
|
| |
|
|
|
| |
Problem: :echowindow does not work in a compiled function.
Solution: Handle the expression at compile time.
|
| |
|
|
|
| |
Problem: Filetype of *.sil files not well detected.
Solution: Inspect the file contents to guess the filetype.
|
| |
|
|
|
| |
Problem: MS-Windows: GUI mouse move event test is flaky.
Solution: Wait for a little while for the first move event.
|
| |
|
|
|
|
| |
Problem: MS-Windows: cannot set cursor shape in Windows Terminal.
Solution: Make cursor shape work with Windows Terminal. (Ken Takata,
closes #11028, closes #6576)
|
| |
|
|
|
|
| |
Problem: :horizontal modifier not fully supported.
Solution: Also use :horizontal for completion and user commands.
(closes #11025)
|
| |
|
|
|
| |
Problem: Error message for list argument could be clearer.
Solution: Include the argument number. (Yegappan Lakshmanan, closes #11027)
|
| |
|
|
|
|
| |
Problem: MS-Windows: background color wrong in Console.
Solution: Figure out the default console background color. (Yasuhiro
Matsumoto, issue #10310)
|
| |
|
|
|
|
|
| |
Problem: ColorScheme autocommand triggered when colorscheme is not found.
(Romain Lafourcade)
Solution: Only trigger ColorScheme when loading the colorscheme succeeds.
(closes #11024)
|
| |
|
|
|
|
| |
Problem: ":wincmd =" equalizes in two directions.
Solution: Make ":vertical wincmd =" equalize vertically only and
":horizontal wincmd =" equalize horizontally only.
|
| |
|
|
|
| |
Problem: mapset() does not restore <Nop> mapping properly.
Solution: Use an empty string for <Nop>. (closes #11022)
|
| |
|
|
|
| |
Problem: The 'cmdheight' zero support causes too much trouble.
Solution: Revert support for 'cmdheight' being zero.
|
