| Commit message (Collapse) | Author | Age |
|\
| |
| |
| |
| |
| | |
clang/"dead assignment": suppessed getchar.c
Possibly invalid, read on line 1972:
if (keylen >= 0 && keylen <= typebuf.tb_len) {
|
| |
| |
| |
| |
| | |
To ensure consistency and avoid possible errors in the code
the assigment is considered to be legit.
|
|\ \
| | |
| | | |
api/lua: add {byte_count} parameter to line region change event
|
| | | |
|
| | | |
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
Regressed in e134cc9d4a: the use of list_in_columns was not adding a
space after the first features, because we do not start on a new line:
> Features: -acl+iconv +tui
This moves all the related code to `list_features`, and just joins them
with spaces.
|
| |
| |
| |
| |
| | |
- use os_getenvname_at_index / os_getenv
- f_getenv: empty (*p == NUL) is not null (undefined)
|
| |
| |
| |
| |
| |
| |
| | |
Problem: There is no easy way to manipulate environment variables.
Solution: Add environ(), getenv() and setenv(). (Yasuhiro Matsumoto,
closes vim/vim#2875)
https://github.com/vim/vim/commit/691ddeefb545d8488e5a495af61caba2e57b3de9
|
| |
| |
| |
| |
| |
| |
| | |
Suppress the warning.
mb_c and mb_l describe a char together, they are not independent variables. The
coupled assignments are good practice to avoid future confusion, even if the
current code doesn't use an assigned value.
|
|/
|
|
|
|
| |
Problem: Json_encode() is very slow for large results.
Solution: In the growarray use a growth of at least 50%. (Ken Takata,
closes vim/vim#4461)
https://github.com/vim/vim/commit/c47ed44be76a520ded90913099771999c8a79eeb
|
|\
| |
| | |
lua: laundry list (crashes and additions)
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
Problem: Warning for size_t/int mixup.
Solution: Change type. (Mike Williams)
https://github.com/vim/vim/commit/d33a764123a8aedb20cd84aeff3b94810ee67c4c
|
| |
| |
| |
| |
| |
| |
| |
| | |
The message is useless, it doesn't even mention the event name.
vim_dev discussion:
https://groups.google.com/forum/#!msg/vim_dev/RTbq58TMq9w/Xr4rSoUTCgAJ
ref: https://github.com/vim/vim/issues/4300
|
| |
| |
| |
| |
| |
| | |
Problem: Aborting an autocmd with an exception is not tested.
Solution: Add a test. Also shows how to abort a command by throwing an
exception.
https://github.com/vim/vim/commit/23b5139234a79567097ca73aba15ea134381b934
|
|/
|
|
|
| |
Problem: No test for completion of mapping keys.
Solution: Add a test. Also clean up the code.
https://github.com/vim/vim/commit/2cb9f0253228478ec27265ff40f59c629874d695
|
| |
|
|
|
|
|
| |
We don't want to retry autoload sourcing (slow) for every random has()
query that finds it way to eval_call_provider().
|
|
|
|
|
| |
Value of 1 cannot be used, because users might set that in their vimrc
to _disable_ a provider, which would confuse :checkhealth and has().
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of deciding provider status in eval_has_provider, move the
decision to the provider Vim scripts.
Previously, provider loading worked as follows:
1. eval_has_provider() verified provider availability by searching for
the provider#providername#Call function and cached this verificaion as a static
variable for some providers
2. providers short-circuited on loading to prevent the definition of the
Call function (with the exception of the node provider that did not)
This commit changes the expected interface between nvim and its
providers to facilitate provider reloading, by splitting the
verification of the provider from the availability of the Call function.
eval_has_provider() now checks for a provider#providername#enabled
variable. It is up to the provider script to set this to 0 or 1
accordingly. eval_call_provider() remains unchanged.
All providers hosting a Call function were updated to respect this.
The clipboard provider now has a Reload function to reload the
provider.
|
|
|
|
|
|
| |
Problem: Error for using "compl", reserved word in C++.
Solution: Rename to "complp". (suggestion by Ken Takata)
https://github.com/vim/vim/commit/52111f82318dfbeb6a0b06230c0609abff509250
|
|
|
|
|
|
| |
Problem: :argdo is not tested
Solution: Add a test.
https://github.com/vim/vim/commit/72e1b39111389001a20fbe8aa344ce2fe0a7e1a4
|
|
|
|
|
|
| |
Problem: Error message may be empty in filetype test.
Solution: Use v:exception instead. (Daniel Hahler, closs vim/vim#4744)
https://github.com/vim/vim/commit/eee9f65b2a213e9031f172d9d3b22adad6cb985d
|
|
|
|
|
|
| |
Problem: Some filetype rules are in the wrong place.
Solution: Move to the right place. Add a few more tests.
https://github.com/vim/vim/commit/c273405188cc2f19e949089cda1f92bd81610dac
|
|
|
|
|
|
| |
Problem: Filetype "vuejs" causes problems for some users.
Solution: Rename to "vue".
https://github.com/vim/vim/commit/4248111497ab78f45d3d48576740949778a1e76b
|
|
|
|
|
|
| |
Problem: Cannot recognize Pipfile.
Solution: Use existing filetypes. (Charles Ross, closes vim/vim#4280)
https://github.com/vim/vim/commit/3a4c53ba51c01b8fea68972453a71c55cf32edbe
|
|
|
|
|
| |
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
|