aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #11908 from bfredl/qf_msg-fixupBjörn Linse2020-02-19
|\ | | | | test: always use "set more" with :digraph test
| * test: always use "set more" with :digraph testBjörn Linse2020-02-19
|/ | | | otherwise we risk the same issue as with ex_cmds/digraphs_spec.lua
* Merge #11895 'lsp: fix textDocument/completion handling'Justin M. Keyes2020-02-18
|\
| * lsp: make functions private and use filter functionHirokazu Hata2020-02-19
| |
| * lsp: respect the sort order if there is sortTextHirokazu Hata2020-02-19
| |
| * lsp: fix textDocument/completion handlingHirokazu Hata2020-02-19
| | | | | | | | | | | | | | fix: #11826 Some lanuguage servers return complementary candidates whose prefixes do not match are also returned. So we exclude completion candidates whose prefix does not match. ex) Microsoft python-language-server, rust-analyzer
| * lua: move test helper function, map and filter, to vim.shared moduleHirokazu Hata2020-02-18
| |
* | clang/scan-build: fix dead stores #11900Hirokazu Hata2020-02-18
| |
* | Merge #11887 'loop_close: close all handles'Justin M. Keyes2020-02-18
|\ \ | |/ |/|
| * loop_close: close all handlesJustin M. Keyes2020-02-17
| | | | | | | | | | | | | | | | | | | | - Move uv_stop(), it still causes a "leak" on exit somehow. - Tenatively restore `UV_RUN_DEFAULT`. It shouldn't hang since we clobber the handles via `uv_walk((h)=>uv_close(h))`. Although this still "leaks" on exit, it's faster than the 2-second timeout. fix #11820 fix #7376
| * loop_close: call uv_stop(), fix bugJustin M. Keyes2020-02-17
|/ | | | | | | | | | | | | | | | | - Call uv_stop(). - Restore `uv_loop_close` condition (braindead cosmetic change from a2efc9cf8b0f that caused uv_loop_close *not* to be called if wait=false, sorry). Not doing `uv_walk(() => uv_close)`: see source comment for explanation. fix #11820 fix #7376 Q: Should we restore use of `UV_RUN_DEFAULT`/`UV_RUN_ONCE` (removed in a2efc9cf8b0f)? A: The while-loop (hopefully) achieves the same purpose while avoiding a hang.
* loop_close: timout after 2 seconds #11821Justin M. Keyes2020-02-16
| | | | | never UV_RUN_DEFAULT ref #11820 ref #7376
* test: always dump logs on failure #11886Justin M. Keyes2020-02-16
| | | | | Whenever `eq()`, `ok()`, etc. fails, include log tail in the failure message. This helps to correlate log messages with a particular test failure.
* Merge #11873 from janlazo/vim-8.1.0786Justin M. Keyes2020-02-16
|\ | | | | vim-patch:8.0.1660,8.1.{43,786,1201,2129,2131,2187,2223,2259},8.2.{241,267}
| * vim-patch:8.2.0267: no check for a following cmd when calling a function failsJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | Problem: No check for a following command when calling a function fails. Solution: Also check for a following command when inside a try block. (closes vim/vim#5642) https://github.com/vim/vim/commit/e51bb17dd0d51384375830ee2a1df30c08121443
| * vim-patch:8.1.0043: ++bad argument of :edit does not work properlyJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | | | Problem: ++bad argument of :edit does not work properly. Solution: Return FAIL from get_bad_opt() only when there is no valid argument. (Dominique Pelle, Christian Brabandt, closes vim/vim#2966, closes vim/vim#2947) https://github.com/vim/vim/commit/7580849df9d6c7c515f5ed784019336d8a8ec0f0
| * vim-patch:8.0.1660: the terminal API "drop" command doesn't support optionsJan Edmund Lazo2020-02-16
| | | | | | | | | | | | Problem: The terminal API "drop" command doesn't support options. Solution: Implement the options. https://github.com/vim/vim/commit/333b80acf3a44e462456e6d5730e47ffa449c83d
| * vim-patch:8.1.1201: output of :command is hard to readJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | Problem: Output of :command is hard to read. Solution: Make some columns wider, some narrower. Truncate the command when listing all. https://github.com/vim/vim/commit/725310d89e1ba268bf410472b7de054c6c260161
| * vim-patch:8.1.2187: error for bad regexp even though regexp is not usedJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | Problem: Error for bad regexp even though regexp is not used when writing a file. (Arseny Nasokin) Solution: Ignore regexp errors. (closes vim/vim#5059) https://github.com/vim/vim/commit/b40c2576d4e0e2dd2c580414c45947d88556d76d
| * vim-patch:8.2.0241: crash when setting 'buftype' to "quickfix"Jan Edmund Lazo2020-02-16
| | | | | | | | | | | | Problem: Crash when setting 'buftype' to "quickfix". Solution: Check that error list is not NULL. (closes vim/vim#5613) https://github.com/vim/vim/commit/99234f29aa8767f2e71bb1f5db6ee0131bc8e64e
| * vim-patch:8.1.2223: cannot see what buffer an ml_get error is forJan Edmund Lazo2020-02-16
| | | | | | | | | | | | Problem: Cannot see what buffer an ml_get error is for. Solution: Add the buffer number and name in the message https://github.com/vim/vim/commit/cb86893114ce33dc9c7bd4ff992b05c12406b35d
| * vim-patch:8.1.0786: ml_get error when updating the status lineJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | | | Problem: ml_get error when updating the status line and a terminal had its scrollback cleared. (Chris Patuzzo) Solution: Check the cursor position when drawing the status line. (closes vim/vim#3830) https://github.com/vim/vim/commit/10772307c4e5299ed45470f92779f089a00d841e
| * vim-patch:8.1.2259: running tests may leave XfakeHOME behindJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | Problem: Running tests may leave XfakeHOME behind. Solution: Source summarize.vim without using setup.vim. (closes vim/vim#5177) Also fix that on MS-Windows the test log isn't echoed. https://github.com/vim/vim/commit/7d2320414ffdd698c9a17d775f71e2c455c21624
| * vim-patch:8.1.2131: MSVC tests failJan Edmund Lazo2020-02-16
| | | | | | | | | | | | Problem: MSVC tests fail. Solution: Replace backslashes with slashes. https://github.com/vim/vim/commit/c25e702deec74771e49f6c2df4cda7a1f97d0c1e
| * vim-patch:8.1.2129: using hard coded executable path in testJan Edmund Lazo2020-02-16
| | | | | | | | | | | | | | Problem: Using hard coded executable path in test. Solution: Use v:progpath. Use $VIMRUNTIME instead of "runtime". (James McCoy, closes vim/vim#5025) https://github.com/vim/vim/commit/1ac41a5c1352306942344777d2ba86dccd84ffad
* | Merge #11837 'LSP: fixes, improve test visibility'Justin M. Keyes2020-02-16
|\ \ | |/ |/|
| * test/LSP: assert contents of log fileJustin M. Keyes2020-02-16
| |
| * lsp/rpc.lua: fix `env` application functionJustin M. Keyes2020-02-16
| | | | | | | | Env vars must be merged with the current env.
| * test/LSP: dump logs on errorJustin M. Keyes2020-02-16
| | | | | | | | | | | | This will help debug CI flakey failures. TODO: helpers.assert_log() -- Explicitly check contents of the logfile.
| * LSP: fix validate_client_configJustin M. Keyes2020-02-16
| | | | | | | | | | - `cmd_env` is a table not a function. - tests: Set $NVIM_LOG_FILE for fake LSP server.
| * test/LSP: use less-generic exit codeJustin M. Keyes2020-02-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - os.exit(1) is too generic, since code 1 may be caused by Nvim exiting for some other reason. Change it to os.exit(101). - style: de-architect json_encode/json_decode calls. Failure seen in travis macOS job: https://travis-ci.org/neovim/neovim/jobs/647849133 [ FAILED ] test/functional/plugin/lsp_spec.lua@ 266 SP basic_init test should not send didOpen if the buffer closes before init test/functional/plugin/lsp_spec.lua:297: exit code Expected objects to be the same. Passed in: (number) 1 Expected: (number) 0 stack traceback: test/functional/plugin/lsp_spec.lua:297: in function 'on_exit' test/functional/plugin/lsp_spec.lua:100: in function 'test_rpc_server' test/functional/plugin/lsp_spec.lua:272: in function <test/functional/plugin/lsp_spec.lua:266>
| * test: styleJustin M. Keyes2020-02-16
| |
| * deps: lua-client 0.2.2-1Justin M. Keyes2020-02-16
|/
* checkhealth: allow 'sudo install' of 'Neovim::Ext' #11874Jan Edmund Lazo2020-02-15
| | | | | | | | | | | | | | cpanm cannot look for Perl modules from root directories without sudo so it creates '~/perl5/' and look for Perl modules in there. Whether this directory existed before running cpanm or not, cpanm returns a warning to advice the user to setup local::lib in order to use modules in '~/perl5/' and exits with error code 0. Each line in the warning always starts with '!'. Display this warning to the user. Continue parsing the version number if the warning can be ignored because lines that are not prefixed with '!' are valid output. Fix #11858
* Merge pull request #11864 from h-michael/deepcopyBjörn Linse2020-02-15
|\ | | | | lua: vim.deepcopy() and vim.tbl_extend() should preserve vim.empty_dict()
| * lua: add vim.tbl_extend and vim.deepcopy testHirokazu Hata2020-02-14
| |
| * lua: if second argument is vim.empty_dict(), vim.tbl_extend uses ↵Hirokazu Hata2020-02-14
| | | | | | | | empty_dict() instead of {}
| * lua: vim.deepcopy uses empty_dict() instead of {} for empty_dict()Hirokazu Hata2020-02-13
| | | | | | | | fix: https://github.com/neovim/nvim-lsp/issues/94
* | mouse.c: can click on multibyte foldopen/foldclose (#11863)Matthieu Coudron2020-02-14
| | | | | | | | | | would previously only work with ascii fillchars. Added a test.
* | Merge pull request #11870 from QnJ1c2kNCg/masterBjörn Linse2020-02-14
|\ \ | | | | | | clang analyzer: Dead assignment `ns_id` in nvim_buf_add_highlight
| * | clang bug: Dead assignment `ns_id`Bruno Roy2020-02-13
| | | | | | | | | | | | | | | | | | | | | Remove a dead assignment of the `ns_id` variable in the `src/nvim/api/buffer.c` file. Refer: https://neovim.io/doc/reports/clang/report-f279da.html#EndPath
* | | build: Fix MSVC build failure on CI #11865erw72020-02-14
| | | | | | | | | | | | | | | clean-shared-libraries does nothing useful in MSVC build. Nevertheless, it deletes ${DEPS_INSTALL_DIR}/lib/nvim/parser/c.dll and causes build failure in CI.
* | | checkhealth: ignore cpamn "!" output #11869Jan Edmund Lazo2020-02-14
|/ / | | | | | | | | | | | | | | cpanm outputs a warning that suggest to use 'sudo' or use local::lib. cpanm exits with 0 so nvim thinks that the command worked. cpanm output that starts with "!" is likely an error. Close #11858
* | doc/lsp: start_client config cmd must be a list (#11866)Mathias Fußenegger2020-02-13
| | | | | | | | | | | | | | doc/lsp: start_client config cmd must be a list After the merge of https://github.com/neovim/neovim/pull/11847 cmd can no longer be a string but must be a list.
* | Merge pull request #11859 from h-michael/empty-dictBjörn Linse2020-02-12
|\ \ | |/ |/| check json_encode behavior for vim.empty_dict()
| * test: add json_encode test for vim.empty_dict()Hirokazu Hata2020-02-12
|/
* Merge #10433 from erw7/vim-8.1.0027Justin M. Keyes2020-02-12
|\ | | | | vim-patch:8.1.{27,32,36,69,70,71,91,92}
| * Fix issue where callbacks are garbage collectederw72020-02-12
| | | | | | | | | | Import necessary part of vim-patch:8.1.1575. https://github.com/vim/vim/commit/75a1a9415b9c207de5a29b25c0d1949c6c9c5c61
| * vim-patch:8.1.0092: prompt buffer test failserw72020-02-12
| | | | | | | | | | | | | | Problem: Prompt buffer test fails. Solution: Set 'nomodified' before closing the window. (Ozaki Kiichi, closes vim/vim#3051 https://github.com/vim/vim/commit/71ef1ba5e996f34d3e0acbe1d89c4c6bfa5e98ba
| * vim-patch:8.1.0091: MS-Windows: Cannot interrupt gdb when program is runningerw72020-02-12
| | | | | | | | | | | | | | Problem: MS-Windows: Cannot interrupt gdb when program is running. Solution: Add debugbreak() and use it in the terminal debugger. Respect 'modified' in a prompt buffer. https://github.com/vim/vim/commit/4551c0a9fcdbdef52836d4852686d54b5e47fdaf