| Commit message (Collapse) | Author | Age |
... | |
| |
| |
| |
| |
| | |
This is clearer/more explicit and avoids potential mismatch between what unibilium thinks vs what Nvim thinks.
For reference: https://github.com/mauke/unibilium/blob/e3b16d6219ca1cb92d98b1d9cc416b49a3ac468e/uniutil.c#L203-L211
|
|\ \
| | |
| | | |
vim-patch:8.1.{1374,1780}
|
| | |
| | |
| | |
| | |
| | |
| | | |
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
|