aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* vim-patch:8.1.0532: cannot distinguish between quickfix and location listShane Smith2019-10-27
| | | | | | Problem: Cannot distinguish between quickfix and location list. Solution: Add an explicit type variable. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/2d67d307ee5dba911e8fbe73193bf596ebf76c1a
* vim-patch:8.1.0488: using freed memory in quickfix codeShane Smith2019-10-27
| | | | | | | 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
* vim-patch:8.1.0469: too often indexing in qf_lists[]Shane Smith2019-10-27
| | | | | | Problem: Too often indexing in qf_lists[]. Solution: Use a qf_list_T pointer. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/108e7b422b7b59153dd5af1fb75e83fa36ff3db4
* vim-patch:8.1.0407: quickfix code mixes using the stack and a list pointerShane Smith2019-10-27
| | | | | | | 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
* vim-patch:8.1.0252: quickfix functions are too longShane Smith2019-10-27
| | | | | | Problem: Quickfix functions are too long. Solution: Refactor. (Yegappan Lakshmanan, closes vim/vim#2950) https://github.com/vim/vim/commit/de3b3677f7eace66be454196db0fbf710cfc8c5e
* Merge #11298 from janlazo/vim-8.1.2220Justin M. Keyes2019-10-26
|\ | | | | vim-patch:8.1.2220
| * ci: do not skip before_install on lint jobJan Edmund Lazo2019-10-26
| |
| * ci: use python3 for flake8Jan Edmund Lazo2019-10-26
| | | | | | | | 'make pylint' fails if flake8 runs on python2.
| * vim-patch:8.1.2220: :cfile does not abort like other quickfix commandsJan Edmund Lazo2019-10-26
|/ | | | | | | 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
* lintJustin M. Keyes2019-10-26
|
* gen_vimdoc.py: dump API docs to msgpack #11296smolck2019-10-26
| | | | | | | | | | | | | | | | | | | | | | | 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
* win/dist: nvim-qt v0.2.15 #11295Justin M. Keyes2019-10-26
|
* Merge #11294 from tomtomjhj/vim-8.1.2173Justin M. Keyes2019-10-26
|\ | | | | vim-patch:8.1.{2173,2207,2218}
| * vim-patch:8.1.2218: "gN" is off by one in Visual modeJaehwang Jerry Jung2019-10-27
| | | | | | | | | | | | 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
| * vim-patch:8.1.2207: "gn" doesn't work quite rightJaehwang Jerry Jung2019-10-27
| | | | | | | | | | | | | | 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
| * vim-patch:8.1.2173: searchit() has too many argumentsJaehwang Jerry Jung2019-10-27
| | | | | | | | | | | | Problem: Searchit() has too many arguments. Solution: Move optional arguments to a struct. Add the "wrapped" argument. https://github.com/vim/vim/commit/92ea26b925a0835badb0af2d5887238a4198cabb
* | tests: remove some redundant legacy tests #11028Daniel Hahler2019-10-26
| | | | | | | | These were turned into new-style Vim tests in cbecae46f.
* | build: add -fstack-protector* to linker #11292vincowl2019-10-26
|/ | | | Needed by chromeOS. fixes #11228
* Merge #11271 from h-michael/add-more-infoJustin M. Keyes2019-10-26
|\ | | | | lua/vim.shared: improve some validation messages
| * test/pcall_err(): truncate full paths, omit linenrJustin M. Keyes2019-10-26
| | | | | | | | ref #11271
| * lua/stdlib: adjust some validation messages #11271Hirokazu Hata2019-10-26
|/ | | | close #11271
* Merge #11275 from janlazo/ci/nodejsJustin M. Keyes2019-10-25
|\ | | | | ci/nodejs: use node v10, prefer shell builtins in ci scripts
| * ci: simplify tree-sitter-cli installJan Edmund Lazo2019-10-25
| | | | | | | | | | Remove useless `npm link` for tree-sitter-cli. Use powershell builtins.
| * CI/Appveyor: set powershell strict modeJan Edmund Lazo2019-10-25
| | | | | | | | | | Set-PSDebug produces too much noise and has global scope. Strict mode is scoped to the script context and catches errors.
| * CI: bump nodejs to v10.x (LTS)Jan Edmund Lazo2019-10-25
| | | | | | | | | | v8.x will be EOL at end of 2019. nvm on Travis has outdated LTS aliases.
* | vim-patch:8.1.0859: handle multibyte "%v" in 'errorformat' #11285Jan Edmund Lazo2019-10-25
| | | | | | | | | | 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
* | man.vim: never switch to non-man window #11286Joshua Rubin2019-10-25
|/ | | | | | | | | | | 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>
* Merge #11213 from janlazo/vim-8.1.0014Justin M. Keyes2019-10-24
|\ | | | | vim-patch:8.1.{14,288,345,410,434,438,455}
| * tag: fix pvs/v547 errorJan Edmund Lazo2019-10-24
| |
| * vim-patch:8.1.0455: checking for empty quickfix stack is not consistentJan Edmund Lazo2019-10-23
| | | | | | | | | | | | Problem: Checking for empty quickfix stack is not consistent. Solution: Use qf_stack_empty(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/019dfe6855e011c02427bb922aafeae0245372c9
| * vim-patch:8.1.0438: the ex_make() function is too longJan Edmund Lazo2019-10-23
| | | | | | | | | | | | Problem: The ex_make() function is too long. Solution: Split it into several functions. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/b434ae2a1fcbbd43244c6130451de7f14346e224
| * vim-patch:8.1.0434: copy_loclist() is too longJan Edmund Lazo2019-10-23
| | | | | | | | | | | | Problem: copy_loclist() is too long. Solution: Split in multiple functions. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/09037503ea5f957ad23121bc61e15e4bb1765edf
| * vim-patch:8.1.0410: the ex_copen() function is too longJan Edmund Lazo2019-10-23
| | | | | | | | | | | | Problem: The ex_copen() function is too long. Solution: Refactor to split off two functions. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/476c0db00205590974395df717519407a7717270
| * vim-patch:8.1.0345: cannot get the window id associated with the location listJan Edmund Lazo2019-10-23
| | | | | | | | | | | | | | 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
| * vim-patch:8.1.0288: quickfix code uses cmdidx too oftenJan Edmund Lazo2019-10-23
| | | | | | | | | | | | Problem: Quickfix code uses cmdidx too often. Solution: Add is_loclist_cmd(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/396659592fe039decc8c088694912067fe32a681
| * vim-patch:8.1.0014: qf_init_ext() is too longJan Edmund Lazo2019-10-23
|/ | | | | | | 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
* Merge #11261 from janlazo/vim-8.1.2185Justin M. Keyes2019-10-22
|\ | | | | vim-patch:8.1.{2185,2188,2190,2197}
| * vim-patch:8.1.2197: ExitPre autocommand may cause accessing freed memoryJan Edmund Lazo2019-10-21
| | | | | | | | | | | | 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
| * vim-patch:8.1.2190: syntax test fails on MacJan Edmund Lazo2019-10-21
| | | | | | | | | | | | Problem: Syntax test fails on Mac. Solution: Limit the window size to 20 rows. https://github.com/vim/vim/commit/83e9a1ce75818a78c5ddf8dcfb820634ca6fabff
| * vim-patch:8.1.2188: build error for missing defineJan Edmund Lazo2019-10-21
| | | | | | | | | | | | | | | | 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.
| * vim-patch:8.1.2185: syntax test failsJan Edmund Lazo2019-10-21
| | | | | | | | | | | | Problem: Syntax test fails. Solution: Add missing file patch. https://github.com/vim/vim/commit/bbfd1562aeaa5b40b6451effc399846b692d6992
* | Merge #11199 from bobrippling/vim-8.1.1228Justin M. Keyes2019-10-22
|\ \ | |/ |/| vim-patch:8.1.{1099,1228,1962} add 'tagfunc'
| * vim-patch:8.1.1962: leaking memory when using tagfunc()Rob Pilling2019-10-22
| | | | | | | | | | | | Problem: Leaking memory when using tagfunc(). Solution: Free the user_data. (Dominique Pelle, closes vim/vim#4886) https://github.com/vim/vim/commit/55008aad50601cae079037fda8fb434cde70c0f4
| * vim-patch:8.1.1228: not possible to process tags with a functionRob Pilling2019-10-22
| | | | | | | | | | | | 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
| * Perform HASHTAB_ITER bookkeeping before user-codeRob Pilling2019-10-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * vim-patch:8.1.1099: the do_tag() function is too longRob Pilling2019-10-12
| | | | | | | | | | | | | | 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
* | ci/install.sh: pin treesitter to v0.15.9 #11266Hirokazu Hata2019-10-20
| | | | | | | | | | 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
* | vim-patch:8.1.2168: heredoc not skipped in if-block #11265Marco Hinz2019-10-20
| | | | | | | | | | | | | | | | 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
* | TUI/thread: guard env map from potential race with unibilium #11259erw72019-10-20
| | | | | | unibi_from_term calls getenv internally, so exclusive control is required.
* | build/doc/CI: remove/update quickbuild references #11258Justin M. Keyes2019-10-19
| |