| Commit message (Collapse) | Author | Age |
|
|
|
|
|
| |
Problem: Cannot distinguish between quickfix and location list.
Solution: Add an explicit type variable. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/2d67d307ee5dba911e8fbe73193bf596ebf76c1a
|
|
|
|
|
|
|
| |
Problem: Using freed memory in quickfix code. (Dominique Pelle)
Solution: Add the quickfix_busy() flag to postpone deleting quickfix lists
until it is safe. (Yegappan Lakshmanan, closes #3538)
https://github.com/vim/vim/commit/9f84ded38b62c82a4ee57b54f403b1b185ed8170
|
|
|
|
|
|
| |
Problem: Too often indexing in qf_lists[].
Solution: Use a qf_list_T pointer. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/108e7b422b7b59153dd5af1fb75e83fa36ff3db4
|
|
|
|
|
|
|
| |
Problem: Quickfix code mixes using the stack and a list pointer.
Solution: Use a list pointer in more places. (Yegappan Lakshmanan,
closes vim/vim#3443)
https://github.com/vim/vim/commit/fe15b7dfa628d4edd683dae9528194c0e5510128
|
|
|
|
|
|
| |
Problem: Quickfix functions are too long.
Solution: Refactor. (Yegappan Lakshmanan, closes vim/vim#2950)
https://github.com/vim/vim/commit/de3b3677f7eace66be454196db0fbf710cfc8c5e
|
|\
| |
| | |
vim-patch:8.1.2220
|
| | |
|
| |
| |
| |
| | |
'make pylint' fails if flake8 runs on python2.
|
|/
|
|
|
|
|
| |
Problem: :cfile does not abort like other quickfix commands.
Solution: Abort when desired. Add tests for aborting. (Yegappan Lakshmanan,
closes vim/vim#5121)
https://github.com/vim/vim/commit/6a0cc916bd3cd6c2fd88b2972c92ade225603229
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Convenient for API clients who want to reuse the API docs in their own
docs. Could be used e.g. to eliminate nvim.net's own doxygen parser:
https://github.com/neovim/nvim.net/tree/3a736232a4e7b7a2a1eff4bded24d2bf27a918c2/src/NvimClient.APIGenerator/Docs
TODO: currently the result values are formatted as Vim help docs. We
should change the values to have structure, something like this:
[{
'nvim_win_get_var': [
'line1,
'line2',
[ 'item1', 'item2', ... ]
],
'nvim_win_set_var': [
...
],
...
}]
close #11296
|
| |
|
|\
| |
| | |
vim-patch:8.1.{2173,2207,2218}
|
| |
| |
| |
| |
| |
| | |
Problem: "gN" is off by one in Visual mode.
Solution: Check moving forward. (Christian Brabandt, vim/vim#5075)
https://github.com/vim/vim/commit/453c19257f6d97904ec2e3823e88e63c983f2f9a
|
| |
| |
| |
| |
| |
| |
| | |
Problem: "gn" doesn't work quite right. (Jaehwang Jerry Jung)
Solution: Improve and simplify the search logic. (Christian Brabandt,
closes vim/vim#5103, closes vim/vim#5075)
https://github.com/vim/vim/commit/edaad6e0a0e3c1fcb6a5c2771e647c52475bb19c
|
| |
| |
| |
| |
| |
| | |
Problem: Searchit() has too many arguments.
Solution: Move optional arguments to a struct. Add the "wrapped" argument.
https://github.com/vim/vim/commit/92ea26b925a0835badb0af2d5887238a4198cabb
|
| |
| |
| |
| | |
These were turned into new-style Vim tests in cbecae46f.
|
|/
|
|
| |
Needed by chromeOS.
fixes #11228
|
|\
| |
| | |
lua/vim.shared: improve some validation messages
|
| |
| |
| |
| | |
ref #11271
|
|/
|
|
| |
close #11271
|
|\
| |
| | |
ci/nodejs: use node v10, prefer shell builtins in ci scripts
|
| |
| |
| |
| |
| | |
Remove useless `npm link` for tree-sitter-cli.
Use powershell builtins.
|
| |
| |
| |
| |
| | |
Set-PSDebug produces too much noise and has global scope.
Strict mode is scoped to the script context and catches errors.
|
| |
| |
| |
| |
| | |
v8.x will be EOL at end of 2019.
nvm on Travis has outdated LTS aliases.
|
| |
| |
| |
| |
| | |
Problem: "%v" in 'errorformat' does handle multi-byte characters.
Solution: Handle multi-byte characters. (Yegappan Lakshmanan, closes vim/vim#3700)
https://github.com/vim/vim/commit/c45eb770a5988734ff2c572e5e2ce307158c33c8
|
|/
|
|
|
|
|
|
|
|
|
| |
In order to find if there was already an open man page, the :Man command
would cycle through each window to see if &ft=='man'. This triggers
autocmds, e.g. BufEnter, unnecessarily and can have unexpected
side-effects.
Change the logic to check each window's ft without switching to it
unless it is actually a man window.
Signed-off-by: Joshua Rubin <me@jawa.dev>
|
|\
| |
| | |
vim-patch:8.1.{14,288,345,410,434,438,455}
|
| | |
|
| |
| |
| |
| |
| |
| | |
Problem: Checking for empty quickfix stack is not consistent.
Solution: Use qf_stack_empty(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/019dfe6855e011c02427bb922aafeae0245372c9
|
| |
| |
| |
| |
| |
| | |
Problem: The ex_make() function is too long.
Solution: Split it into several functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/b434ae2a1fcbbd43244c6130451de7f14346e224
|
| |
| |
| |
| |
| |
| | |
Problem: copy_loclist() is too long.
Solution: Split in multiple functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/09037503ea5f957ad23121bc61e15e4bb1765edf
|
| |
| |
| |
| |
| |
| | |
Problem: The ex_copen() function is too long.
Solution: Refactor to split off two functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/476c0db00205590974395df717519407a7717270
|
| |
| |
| |
| |
| |
| |
| | |
Problem: Cannot get the window id associated with the location list.
Solution: Add the "filewinid" argument to getloclist(). (Yegappan
Lakshmanan, closes vim/vim#3202)
https://github.com/vim/vim/commit/c9cc9c78f21caba7ecb5c90403df5e19a57aa96a
|
| |
| |
| |
| |
| |
| | |
Problem: Quickfix code uses cmdidx too often.
Solution: Add is_loclist_cmd(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/396659592fe039decc8c088694912067fe32a681
|
|/
|
|
|
|
|
| |
Problem: qf_init_ext() is too long.
Solution: Split it into multiple functions. (Yegappan Lakshmanan,
closes vim/vim#2939)
https://github.com/vim/vim/commit/6053f2d29a979ffed1fe01b0a2f28e23750530e9
|
|\
| |
| | |
vim-patch:8.1.{2185,2188,2190,2197}
|
| |
| |
| |
| |
| |
| | |
Problem: ExitPre autocommand may cause accessing freed memory.
Solution: Check the window pointer is still valid. (closes vim/vim#5093)
https://github.com/vim/vim/commit/34ba06b6e6f94bb46062e6c85dbfdcbb0d255ada
|
| |
| |
| |
| |
| |
| | |
Problem: Syntax test fails on Mac.
Solution: Limit the window size to 20 rows.
https://github.com/vim/vim/commit/83e9a1ce75818a78c5ddf8dcfb820634ca6fabff
|
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: Build error for missing define.
Solution: Add missing change.
https://github.com/vim/vim/commit/2b78ab5d0c91c229715ae140a34978506343bde3
These "WILD_" macros are used in earlier vim patches.
|
| |
| |
| |
| |
| |
| | |
Problem: Syntax test fails.
Solution: Add missing file patch.
https://github.com/vim/vim/commit/bbfd1562aeaa5b40b6451effc399846b692d6992
|
|\ \
| |/
|/| |
vim-patch:8.1.{1099,1228,1962} add 'tagfunc'
|
| |
| |
| |
| |
| |
| | |
Problem: Leaking memory when using tagfunc().
Solution: Free the user_data. (Dominique Pelle, closes vim/vim#4886)
https://github.com/vim/vim/commit/55008aad50601cae079037fda8fb434cde70c0f4
|
| |
| |
| |
| |
| |
| | |
Problem: Not possible to process tags with a function.
Solution: Add tagfunc() (Christian Brabandt, Andy Massimino, closes vim/vim#4010)
https://github.com/vim/vim/commit/45e18cbdc40afd8144d20dcc07ad2d981636f4c9
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The `HASHTAB_ITER` logic keeps track of how many entries in the hash
table are left to visit, decrementing this on each iteration of the
loop. This was previously decremented at the end of the loop body:
```c
size_t hi##todo_ = hi##ht_->ht_used;
for (hashitem_T *hi = hi##ht_->ht_array; hi##todo_; hi++) {
if (!HASHITEM_EMPTY(hi)) {
{
<user code, macro-inserted>
}
hi##todo_--; // <--- important decrement here
}
}
```
This meant that if the body of the loop (substituted in via macro
expansion) contained a `continue` statement, we'd skip decrementing our
counter, meaning we'd iterate too many times over the hash table,
usually leading to an out of bounds read beyond the hash table's
memory, or uninitialised/null pointers from unused hash table slots.
Decrementing `hi##todo` before the arbitrary loop body protects us from
this, and has no adverse side-effects since only the macro code can (or
should) use this variable.
Before this commit, no code within `HASHTAB_ITER()` contained a
`continue`, meaning this bug was left dormant and the fix has a very
minimal chance of introducing any bugs.
|
| |
| |
| |
| |
| |
| |
| | |
Problem: The do_tag() function is too long.
Solution: Factor parts out to separate functions. Move simplify_filename()
to a file where it fits better. (Andy Massimino, closes vim/vim#4195)
https://github.com/vim/vim/commit/b4a6020ac6a0638167013f1e45ff440ddc8a1671
|
| |
| |
| |
| |
| | |
When "tree-sitter test" is executed, query test is also executed, but "tree-sitter-c" does not have query test yet, so cli version that does not include query test execution To use.
ref https://github.com/tree-sitter/tree-sitter/commit/e14e285a1087264a8c74a7c62fcaecc49db9d904
|
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: Heredoc assignment not skipped in if block.
Solution: Check if "skip" is set.
https://github.com/vim/vim/commit/b1ba9abcb385b0a5355788a7eefef78ec68d2f65
Fixes https://github.com/neovim/neovim/issues/11264
|
| |
| |
| | |
unibi_from_term calls getenv internally, so exclusive control is required.
|
| | |
|