| Commit message (Collapse) | Author | Age |
... | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: In tests, when WaitFor() fails it doesn't say why. (James McCoy)
Solution: Add WaitForAssert(), which produces an assert error when it fails.
https://github.com/vim/vim/commit/50182fa84e20a0547f3e2bd6683ef799fcd27855
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Incomplete testing for completion fix. (Lifepillar)
Solution: Add a test with CTRL-P.
https://github.com/vim/vim/commit/bad0ce7b26be5eed8524347018f4c835b212f8d1
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: No test for the popup menu positioning.
Solution: Add a screendump test for the popup menu.
https://github.com/vim/vim/commit/6bb2cdfe604e51eec216cbe23bb6e8fb47810347
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Timer causes error on exit from Ex mode. (xtal8)
Solution: save and restore the ex_pressedreturn flag. (Christian Brabandt,
closes vim/vim#2079)
https://github.com/vim/vim/commit/f5291f301e9322545f0621b2157e93050d1d4fb3
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Function for ex command is named inconsistently.
Solution: Rename do_marks() to ex_marks().
https://github.com/vim/vim/commit/4bd782339e370bde82c2a8976df9f335cc12eba9
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Compiler test for Perl may fail.
Solution: Accept any error line number. (James McCoy, closes vim/vim#4944)
https://github.com/vim/vim/commit/cebfcffa40c058119bc2f92f0db02dffd3f6affe
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: MS-Windows: compiler test may fail when using %:S.
Solution: Reset 'shellslash'.
https://github.com/vim/vim/commit/dff2adc8ddcb6c8f3390a82c321362f8d6756fb8
|
| | |/
| | |
| | |
| | |
| | | |
Problem: assert_fails() does not take a message argument
Solution: Add the argument.
https://github.com/vim/vim/commit/1307d1c003b01b4f67524c95feb07c3d91c7c428
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Before 6e9ea5adc `win_ins_lines` would return `FAIL` for `i/line_count == 0`.
Handle this by checking it in the outer `if`.
Ref: https://github.com/neovim/neovim/commit/6e9ea5ad#commitcomment-35084669
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Assert functions don't return anything.
Solution: Return non-zero when the assertion fails.
https://github.com/vim/vim/commit/65a5464985f980d2bbbf4e14d39d416dce065ec7
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Cannot write and read terminal screendumps.
Solution: Add term_dumpwrite(), term_dumpread() and term_dumpdiff().
Also add assert_equalfile().
https://github.com/vim/vim/commit/d96ff165113ce5fe62107add590997660e3d4802
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Zero out the caller's FileInfo in all the functions which populate the
struct. The contents are considered private, so we need to ensure
it's initialized.
If the stat call fails, the buffer we get back from libuv may not have
any valid data in it, so don't copy it into the caller's FileInfo.
This was happening, expectedly, in functional/ex_cmds/write_spec.lua's
"write errors out correctly", which caused it to fail in certain
environments:
test/functional/ex_cmds/write_spec.lua:130: Expected objects to be the same.
Passed in:
(string) 'Vim(write):E212: Can't open file for writing: not a directory'
Expected:
(string) 'Vim(write):E166: Can't open linked file for writing'
|
|/
|
|
|
|
|
|
|
|
| |
Vim's documentation simply states that libcallnr() should be used "for a
function that returns an int". Based on the tests, code, and common
syscall interfaces, this should likely be taken literally instead of
trying to apply some well-defined type lipstick.
Notably, this change fixes Test_libcall_libcallnr on hppa (a 32-bit
big-endian system).
|
|\
| |
| | |
compositor: avoid transmitting invalid lines on double scroll
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This happens in an operation which both increases topline and also
inserts new lines somewhere in the remaining are. So before drawing any
line, win_update() is performing two grid_scroll operations.
===
A
B
C
D
E^
F
===
Consider that new line will be inserted after line E and screen also
scrolled up to line C. First the topline will be adjusted (x is the
scrolling region, ! invalid/empty space created by the scroll):
===
C x
D x
E^ x
F x
! x
! x
===
and then space is inserted for the new line
===
C
D
E^
! x
F x
! x
===
The problem is that we are now assuming that any invalid area ! created
by a scroll is filled with actual contents (by win_line etc) before it
is scrolled again. But in this case the last invalid line ! gets
scrolled. Ideally we should make win_update smarter and just scroll
valid lines for the later scroll (it is just wasteful to scroll the
larger area anyway), but for the 0.4 releasejust make
the compositor ignore such an invalid line (as it will get overdrawn
anyway later).
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Set the environment variables before changing $HOME
so that python,ruby can find user modules not in $HOME.
Reference:
- https://github.com/neovim/neovim/commit/7be7ec98a22451ec75ce5eb9284fa1722b53c697
- https://github.com/neovim/neovim/issues/10270#issuecomment-531516948
|
|/
|
|
|
|
|
|
|
| |
Problem: Ruby converts v:true and v:false to a number.
Solution: Use Qtrue and Qfalse instead. (Masataka Pocke Kuwabara,
closes vim/vim#3259)
https://github.com/vim/vim/commit/d84b26a03b13cd816d80ff32b61e8de740d499ce
nvim does not support v:none.
|
|
|
|
|
| |
Given the other type names "jumps", "vars", etc., the name "buflist"
is somewhat unintuitive.
|
|
|
|
|
| |
Since the parameter is already non-primitive, make it an `opts` map
instead of just a list, in case we want to extend it later.
|
|\
| |
| |
| | |
vim-patch:8.0.{1134,1148},8.1.{212,289,2028}
fix #10270
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: Preferred cursor column not set in interfaces.
Solution: Set w_set_curswant when setting the cursor. (David Hotham,
closes vim/vim#3060)
https://github.com/vim/vim/commit/53901442f37a59e5495165f91db5574c0b43ab04
This contains test_ruby changes only.
test_python changes were ported before.
|
| |
| |
| |
| | |
These tests didn't pass in https://github.com/neovim/neovim/pull/9245
|
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: Segmentation fault when Ruby throws an exception inside :rubyfile
command.
Solution: Use rb_protect() instead of rb_load_protect(). (ujihisa,
closes vim/vim#2147, greywolf, closes vim/vim#2512, vim/vim#2511)
https://github.com/vim/vim/commit/37badc898b8d167e11553b6d05908ffd35928a6e
|
| |
| |
| |
| |
| |
| | |
Problem: Superfluous call to syn_get_final_id().
Solution: Remove it. (Ken Takata)
https://github.com/vim/vim/commit/76301956f0c079d893cfd927b11456328bed4f9b
|
| |
| |
| |
| |
| |
| | |
Problem: Options test script does not work.
Solution: Use optiondefs.h for input.
https://github.com/vim/vim/commit/d9b0d83b13d2691e4544709abd87eac004715175
|
| |
| |
| |
| |
| |
| | |
Problem: Cursor moves to wrong column after quickfix jump.
Solution: Set the curswant flag. (Andy Massimino, closes vim/vim#3331)
https://github.com/vim/vim/commit/2dfcef4c08a3371e2126504bea00b274f937a840
|
|/
|
| |
fix #8428
|
|
|
|
|
| |
Problem: No test for synIDattr() returning "strikethrough".
Solution: Extend the synIDattr() test. (Jaskaran Singh, closes vim/vim#4929)
https://github.com/vim/vim/commit/dcb2b9cb085288d2a2e4cc31abc591ade04e782e
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
During a refactor long ago, we changed the `getdigits_*` familiy of
functions to abort on overflow. But this is often wrong, because many
of these codepaths are handling user input.
Solution:
Decide at each call-site whether to use "strict" mode.
fix #5555
|
| |
|
|
|
|
|
| |
Problem: Too many #ifdefs.
Solution: Always build with the +visualextra feature.
https://github.com/vim/vim/commit/870ba5f6dce974b6c1c47bf9d3b20db805d10b36
|
|
|
|
|
| |
Problem: Possibly using uninitialized memory.
Solution: Check if "dict" is NULL. (closes vim/vim#4925)
https://github.com/vim/vim/commit/dbec74907eb614517bcf27116d5aad32e087f241
|
|
|
|
| |
Closes https://github.com/neovim/neovim/issues/10031
|
|
|
|
|
|
| |
fix #3436
Includes:
vim-patch:8.0.1038: strike-through text not supported
|
|
|
|
|
|
| |
Problem: No good check if restoring quickfix list worked.
Solution: Let qf_restore_list() return OK/FAIL. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/90f1e2b7bcf56112e1535b693acf131727179a6e
|
|
|
|
|
|
| |
Problem: Coverity complains about a negative array index.
Solution: When qf_id2nr() cannot find the list then don't set qf_curlist.
https://github.com/vim/vim/commit/38efd1d17a6e6aa2add71efdf2cde4a788e5f5e5
|
|
|
|
|
|
| |
Problem: No test for fixed quickfix issue.
Solution: Add a test. Clean up the code a bit. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/3f347e4716c44cf6458be407e712e3d708d82580
|
|
|
|
|
|
| |
Problem: duplicated quickfix code.
Solution: Move the code to a function.
https://github.com/vim/vim/commit/8d8a65e389cef318ae661ff0fe7b1b00fd7cb25f
|
|
|
|
|
|
| |
Problem: Quickfix: mixup of FALSE and FAIL, returning -1.
Solution: Use FAIL and INVALID_QFIDX. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/29ce409bfca52bb8a07e2975d06fd788458e9861
|
|
|
|
|
|
|
|
|
| |
Before this, --embed UIs (without --headless) would not trigger UIEnter.
For TUI, maybe UIEnter isn't useful, but:
- It is less "surprising"/special.
- Makes documentation simpler.
- When TUI becomes a coprocess, it will happen anyway.
|
|
|
|
|
| |
"enter"/"leave" is more conventional for Vim events, and
"attach"/"detach" distinction does not gain much.
|
|
|
|
|
| |
Even though it's always zero currently, it's less confusing if all UIs
have the same fields.
|
|
|
|
|
| |
doc: ginit.vim, gvimrc
fix #3656
|
| |
|
| |
|
| |
|
| |
|