| Commit message (Collapse) | Author | Age |
... | |
| | |
| | |
| | |
| | |
| | |
| | | |
Patch 8.1.1379 includes a change to a test, added in Patch 8.1.0815.
Patch 8.1.0815 includes runtime doc changes to Blob.
Neovim does not support Blobs yet.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Check for file changed triggers too often.
Solution: Don't use "b_p_ar" when it is negative.
https://github.com/vim/vim/commit/c97582b0296cb6f63f3c2e5a0eb954f5920a8e42
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Warning for file no longer available is repeated every time Vim is
focused. (Brian Armstrong)
Solution: Only give the message once. (closes vim/vim#4748)
https://github.com/vim/vim/commit/674e2bde6e7b0c468f304713aa8f97a45e1fcc89
|
| | | |
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- process_stop: do not close stdin explicitly. The "close stdin" step was from
aa9cb48, before we fixed/reworked the SIGTERM timing logic. So it's probably
outdated / no longer needed.
- win: jobstop: exit_code 15
GetExitCodeProcess appears to return the used signal.
https://docs.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-getexitcodeprocess
ref #10573
|
|\ \
| | |
| | | |
clang/"null pointer dereference": ex_cmds.c
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Stopping a job does not work properly on OpenBSD.
Solution: Do not use getpgid() to check the process group of the job
processs ID, always pass the negative process ID to kill().
(George Koehler, closes vim/vim#3656)
https://github.com/vim/vim/commit/76ab4fd61901090e6af3451ca6c5ca0fc370571f
Ref: https://github.com/neovim/neovim/issues/9704
Ref: https://github.com/neovim/neovim/issues/10182#issuecomment-514450069
Closes https://github.com/neovim/neovim/pull/10660
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
It should be done for timeouts only (-1, not != -1), but the job is
stopped via `:bwipeout!` already also in that case.
Fixes E900 with timing out tests:
"test_xxx.tlog" "test_xxx.tlog" [New] 183L, 13375C written
Error detected while processing function Main:
line 37:
E900: Invalid channel id================================================================================
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Not sure what happened in daca22f5b / #8821, but Vim used assert_match /
no expand in vim/vim@d19b23415 (8.1.0005) already.
|
|/ / |
|
|\ \
| | |
| | | |
vim-patch:8.1.{1005,1041,1049,1052,1053,1086,1394}
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Too many curly braces.
Solution: Remove curly braces where they are not needed. (Hirohito Higashi,
closes vim/vim#3982)
https://github.com/vim/vim/commit/abab0b0fdd6535969447b03a4fffc1947918cf6c
Neovim code style requires the opposite.
Add curly braces to minimize lint errors when applying Vim patches.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: When user tries to exit with CTRL-C message is confusing.
Solution: Only mention ":qa!" when there is a changed buffer. (closes vim/vim#4163)
https://github.com/vim/vim/commit/a84a3dd6635fcd2e07f510cba6a999585dcc381a
vim-patch:8.1.1052: test for CTRL-C message sometimes fails
Problem: test for CTRL-C message sometimes fails
Solution: Make sure there are no changed buffers.
https://github.com/vim/vim/commit/553e5a5c568e7d175b65b0472cd6d9843b25f4c8
vim-patch:8.1.1053: warning for missing return statement
Problem: Warning for missing return statement. (Dominique Pelle)
Solution: Add return statement.
https://github.com/vim/vim/commit/d6c3f1fa2b5e1dd7dc87cf608d72b84ad696b58f
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Test for Arabic no longer needed.
Solution: Remove the test for something that was intentionally left out.
https://github.com/vim/vim/commit/295e3ba31f76818f195c654df506c88cb65f9289
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Not restoring t_F2 in registers test.
Solution: Assign to &t_F2 instead of t_F2. (Andy Massimino, closes vim/vim#4434)
https://github.com/vim/vim/commit/a5e44600ebb02057fb15ae7a2f15c5cae8f63bcc
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Test fails because t_F2 is not set.
Solution: Add try-catch.
https://github.com/vim/vim/commit/55d81cd2a15d93dba89b2ff1c988d82789919b6b
|
|\ \ \
| | | |
| | | | |
fix #10604
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: unreliable types for complete function arguments
Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi)
vim/vim#2993
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: unreliable types for complete function arguments
Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi)
vim/vim#2993
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: unreliable types for complete function arguments
Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi)
vim/vim#2993
|
| | | |
| | | |
| | | |
| | | | |
The pointer does not outlive the value in local scope as it gets
reassigned immediately after scope.
|
| |/ /
|/| | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* if braces
* spaces
* post increments
* comment style
|
| | |
| | |
| | |
| | | |
makemap: same condition is used to skip outer loop cycle
|
| | |
| | |
| | |
| | |
| | |
| | | |
(This was originally a Neovim patch, but this commit merges some changes
from the Vim patch.)
https://github.com/vim/vim/commit/d7f246c68cfb97406bcd4b098a2df2d870b3ef92
|
| | |
| | |
| | |
| | |
| | | |
Problem: Shell command completion has duplicates. (Yegappan Lakshmanan)
Solution: Use a hash table to avoid duplicates. (Ozaki Kiichi, closes vim/vim#539, closes vim/vim#2733)
https://github.com/vim/vim/commit/62fe66f251263715968442e237742d9d3dfd5fa1
|
| | |
| | |
| | |
| | | |
Also revert possibly-wrong change in f_sign_getplaced() from
652be3cb0040.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: delete-with-register dispatches TextYankPost before updating yank registers
Solution: Add flag to op_yank().
Fixes #10225
|
|\ \ \
| | | |
| | | | |
vim-patch:8.1.{990,992,1765}
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Floating point exception with "%= 0" and "/= 0".
Solution: Avoid dividing by zero. (Dominique Pelle, closes vim/vim#4058)
https://github.com/vim/vim/commit/e21c1580b7acb598a6e3c38565434fe5d0e2ad7a
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: get(func, dict, def) does not work properly.
Solution: Handle NULL dict better. (Takuya Fujiwara, closes vim/vim#4734)
https://github.com/vim/vim/commit/f91aac5e3e3b8b1633d84eac2687ebbd76d8133b
|
| | | |
| | | |
| | | |
| | | | |
Functions marked with FUNC_ATTR_NONNULL_RET do not return NULL.
Remove redundant checks.
|
|/ / /
| | |
| | |
| | |
| | | |
problem: bfredl: pvs thinks the type of the pointed
at item is too small.
solution: refactored address calculation.
|
| | |
| | |
| | | |
* shorten_buf_name: condition inside outer if-branch with inverse condition
|
| | |
| | |
| | |
| | |
| | |
| | | |
vim-patch:8.0.1753: various warnings from a static analyser
Problem: Various warnings from a static analyser
Solution: Remove unneeded conditions.
https://github.com/vim/vim/commit/1c17ffa4611f4efe68c61f7cdd9ed692a866ba75
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Check shada_pack_xx() result, abort on failure.
- Use xcalloc() to zero-initialize `wms`, fixes below ASAN failure.
ASAN failure (running vim_spec.lua in a loop):
../src/nvim/shada.c:1773:13: runtime error: load of value 224, which is not a valid value for type 'bool'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../src/nvim/shada.c:1773:13
Adding an explicit check just before the ASAN failed line:
case kSDItemRegister: {
if (*((int *)(void *)&entry.data.reg.is_unnamed) != 1
&& *((int *)(void *)&entry.data.reg.is_unnamed) != 0) {
abort();
(gdb) p entry.data.reg
+p entry.data.reg
$5 = {name = 49 '1', type = kMTLineWise, contents = 0x60200000c250, is_unnamed = true, contents_size = 1, width = 0, additional_data = 0x0}
(gdb) p *((uint8_t *)(void *)&entry.data.reg.is_unnamed)
+p *((uint8_t *)(void *)&entry.data.reg.is_unnamed)
$7 = 2049
|
| | | |
|
| | | |
|
|\ \ \ |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: No mode char for terminal mapping from maparg().
Solution: Check for TERMINAL mode. (closes vim/vim#4735)
https://github.com/vim/vim/commit/14371ed69778107654e39268d0d90982e53ad6e0
|
| | | |
| | | |
| | | | |
earlier condition leads to short-circuit of the loop
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* before changed line status variable is set to RA_FAIL in 2 places
line 3816 and 3826, both lead to short-circuit of the loop, making
comparing it to RA_FAIL useless
close #10624
|
|/ / /
| | |
| | |
| | | |
Since timed_out is pointer to int condition timed_out != NULL && timed_out makes no sense as both are effectively the same thing. Most likely the first check was meant to guard the dereference of the second.
Wrong import of Vim patch (8.0.0643).
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: :args output is not aligned.
Solution: Output a line break after the last item in a row.
https://github.com/vim/vim/commit/74da39373c90fcb390068903b5bbb93ce7ac16fa
vim-patch:8.1.1750: depending on the terminal width :version may miss a line break
Problem: Depending on the terminal width :version may miss a line break.
Solution: Add a line break when needed.
https://github.com/vim/vim/commit/8a5c29aee978345132ad7f318b8a84633c33905c
vim-patch:8.1.1760: extra line break for wrapping output of :args
Problem: Extra line break for wrapping output of :args.
Solution: Avoid the extra line break. (Daniel Hahler, closes vim/vim#4737)
https://github.com/vim/vim/commit/9800bfe0fc7596e8fee97172139f0777bac639fb
|