aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* tests: fix/improve "TUI background color" tests (#10229)Daniel Hahler2019-07-03
| | | | - Ensure the opposite of the expected bg is active. - Improves performance: 1.2s instead of 4.5s.
* ci: Travis: do not close fold on failure [skip ci]Daniel Hahler2019-07-03
| | | | | This makes it easier to inspect the test failure, without having to open two folds (typically), and scrolling around.
* build: Fix rule of `build/.ran-third-party-cmake` #10402Eisuke Kawashima2019-07-03
| | | In order that `$(DEPS_BUILD_DIR)/Makefile` exists under parallel make
* tests: fix flaky "TermClose event triggers when fast-exiting terminal job ↵Daniel Hahler2019-07-02
| | | | | | | | | | | stops" (#10377) [ ERROR ]...neovim/neovim/test/functional/autocmd/termclose_spec.lua @ TermClose event triggers when fast-exiting terminal job stops test/functional/helpers.lua:96: Vim(call):E900: Invalid channel id stack traceback: test/functional/helpers.lua:96: in function 'request' test/functional/helpers.lua:254: in function 'command' ...neovim/neovim/test/functional/autocmd/termclose_spec.lua:23: in function <...neovim/neovim/test/functional/autocmd/termclose_spec.lua:20>
* tests: fix flaky "terminal (with fake shell) with not arguments …" (#10401)Daniel Hahler2019-07-02
| | | | | | `retry()` would only try it three times (waiting for 20ms in between), despite the large timeout. Fixes https://github.com/neovim/neovim/issues/10265.
* Merge pull request #10397 from bfredl/cmdcleanupBjörn Linse2019-07-02
|\ | | | | cmdline: remove local variables i and j from command_line_state
| * cmdline: remove local variables i and j from command_line_stateBjörn Linse2019-07-02
|/
* defaults: wildoptions=pum,tagfile #10384Justin M. Keyes2019-07-01
| | | ref #6289
* test/old: pass Test_recover_root_dir on Windows (#10207)Jan Edmund Lazo2019-07-01
| | | | | | | | | | | | | * Revert "oldtests: win: keep set dir=/" This reverts commit 714e0f8bf05cc6febcbcd8cbcbe5f119c37a14b0. * test/old: set swap directory to non-existent drive Appveyor uses Administrator user so it can write anywhere in C:\\. Neovim creates a directory for the swap file if the directory does not exist. D:\\ is the backup/recovery drive so use F:\\ invoke error E303.
* Merge pull request #10389 from bfredl/luafloatBjörn Linse2019-07-01
|\ | | | | api/lua: make nvim_execute_lua use native lua floats, not special tables
| * api/lua: make nvim_execute_lua use native lua floats, not special tablesBjörn Linse2019-07-01
| | | | | | | | | | | | | | | | | | | | Rationale: the purpose of nvim_execute_lua is to simply call lua code with lua values. If a lua function expects a floating point value, it should be enough to specify a float as argument to nvim_execute_lua. However, make sure to preserve the existing roundtripping behavior of API values when using `vim.api` functions. This is covered by existing lua/api_spec.lua tests.
* | vim-patch:8.1.0452: MS-Windows: not finding intl.dll #10388Jan Edmund Lazo2019-07-01
| | | | | | | | | | Problem: MS-Windows: not finding intl.dll. Solution: Also find intl.dll next to libintl.dll. (Ken Takata) https://github.com/vim/vim/commit/7554c548a493cba50b2d0ea3521cac14f28a1f07
* | Merge pull request #10387 from bfredl/wildposBjörn Linse2019-07-01
|\ \ | |/ |/| cmdline: correct the column position of wildoptions=pum popupmenu
| * cmdline: correct the column position of wildoptions=pum popupmenuBjörn Linse2019-07-01
|/ | | | | - position might get invalid with "longest" match - position might be wrong when completed pattern ends in "/"
* build: fix warning with passively available libintl (#10380)Daniel Hahler2019-07-01
| | | Since 0364f51 it would display a warning.
* build: LibLUV: update required version [ci skip] (#10381)Daniel Hahler2019-07-01
| | | | | | | Required after d33aaa0f5. Does not really make a difference, since the VERSION is not handled with our FindLibLUV (due to missing pkg-config information (https://github.com/luvit/luv/issues/354)).
* Merge pull request #10316 from bfredl/cb_safetyBjörn Linse2019-06-30
|\ | | | | luv callbacks: throw error on deferred methods instead of crashing
| * make vim.loop == require'luv'Björn Linse2019-06-30
| | | | | | | | | | This avoids initializing libluv a second time if a plugin invokes require'luv'. It is probably not an issue, but better to be safe.
| * libluv: use luv_set_callback to control callback executionBjörn Linse2019-06-30
| | | | | | | | | | Disable the use of deferred API functions in a fast lua callback Correctly display error messages from a fast lua callback
| * rename: FUNC_API_ASYNC => FUNC_API_FASTBjörn Linse2019-06-30
| |
* | fileio.c: eliminate set_file_time() #10357Justin M. Keyes2019-06-30
| | | | | | Introduce os_file_settime(), remove cruft.
* | man.vim: Handle ANSI escape sequences with ":" #10267Kovas Palunas2019-06-30
|/ | | | closes #10267
* :digraphs : check for CTRL-C less often #10376Justin M. Keyes2019-06-30
| | | No need to do this so frequently.
* Makefile: CMAKE_INSTALL_PREFIX: skip parsing CMAKE_EXTRA_FLAGS if set (#10374)Daniel Hahler2019-06-30
| | | | | Fixes: > Recursive variable 'CMAKE_EXTRA_FLAGS' references itself (eventually). Stop.
* build: update cmake/LibFindMacros.cmake (#10355)Daniel Hahler2019-06-30
| | | Source: https://github.com/Tronic/cmake-modules/blob/d6b5e94625d41469eaf1e2c484d7204cd263893d/LibFindMacros.cmake
* Makefile: move `all` target to the top [ci skip] #10375Daniel Hahler2019-06-30
| | | Fixes "make: Nothing to be done for 'checkprefix'." after 5031e3298.
* doc [ci skip] #10177Justin M. Keyes2019-06-30
| | | ref #10278 #10279 #10353
* build: update some test dependencies (#10339)Daniel Hahler2019-06-29
| | | | | | | | | | * build: update some test dependencies * luacheck ignores * BuildLua: add ${BUSTED} to depends for ${BUSTED_LUA} This is required to rebuild it when busted gets updated.
* build: FindLibLUV: use PkgConfig (#10359)Daniel Hahler2019-06-29
| | | | | | Apparently forgotten in c83926cd0. - Also LIMIT_SEARCH was never used. - Also add LIBLUV_DEFINITIONS, as used with other Find modules.
* Makefile: revisit/improve checkprefix handling (#10348)Daniel Hahler2019-06-29
| | | | | | | | | | | | | | Main improvement: do not error out, but re-run CMake in case CMAKE_INSTALL_PREFIX changed, and only check it for "install". - only look at CMAKE_EXTRA_FLAGS via shell if not empty - add CMAKE_INSTALL_PREFIX to CMAKE_EXTRA_FLAGS (not CMAKE_FLAGS), to override it being set in CMAKE_EXTRA_FLAGS from local.mk - use an empty "checkprefix" target if CMAKE_INSTALL_PREFIX is not provided - skip checking of cached value without build/.ran-cmake; it will be run then anyway - only use it with "install" target; it is only relevant there - do not error, but re-run CMake (by removing the stamp file)
* :ls : show "R", "F" for terminal-jobs #10370Justin M. Keyes2019-06-29
| | | | | | | | | | | This matches Vim behavior. From `:help :ls` : R a terminal buffer with a running job F a terminal buffer with a finished job ? a terminal buffer without a job: `:terminal NONE` TODO: implement `:terminal NONE`. ref #10349
* build: CMake: do not set CMP0059 to old (#10363)Daniel Hahler2019-06-29
| | | | | | | | | Keeps using add_definitions for compatibility with older CMake. Newer CMake (3.12) would have `add_compile_definitions`, but it is not required, since `add_defitions` was meant to be used for compile/preprocessor definitions initially anyway. Ref: https://github.com/neovim/neovim/pull/4389
* cmake/RunTests.cmake: fix TEST_TAG/TEST_FILTER [ci skip] #10371Daniel Hahler2019-06-29
| | | | | Followup to f1f9a2f97. The mentioned example there did not work after all, likely due to changing quoting during PR review.
* tests: fix flaky "timers can be stopped from the handler" (#10364)Daniel Hahler2019-06-29
| | | | | | | | | | | | Seen on Travis (osx): [ RUN ] timers can be stopped from the handler: FAIL .../build/neovim/neovim/test/functional/eval/timer_spec.lua:167: Expected objects to be the same. Passed in: (number) 2 Expected: (number) 3 stack traceback: .../build/neovim/neovim/test/functional/eval/timer_spec.lua:167: in function <.../build/neovim/neovim/test/functional/eval/timer_spec.lua:153>
* viml/profile: switch to uv_gettimeofday() #10356Justin M. Keyes2019-06-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Performance of high-resolution time (clock_gettime via uv_hrtime) is expensive on some systems. For profiling VimL, syntax, etc., we don't care about nanosecond-precision and monotonicity edge-cases, so avoid uv_hrtime(). closes #10328 From the uv__hrtime() source: https://github.com/libuv/libuv/blob/0cdb4a5b4b706d0e09413d9270da28f9a88dc083/src/unix/linux-core.c#L442-L462 /* Prefer CLOCK_MONOTONIC_COARSE if available but only when it has * millisecond granularity or better. CLOCK_MONOTONIC_COARSE is * serviced entirely from the vDSO, whereas CLOCK_MONOTONIC may * decide to make a costly system call. */ This micro-benchmark (Debug build) shows negligible differences on my system: #include <sys/time.h> ... proftime_T tm = profile_start(); int trials = 999999; int64_t t = 0; struct timeval tv; for (int i = 0; i < trials; i++) { t += gettimeofday(&tv,NULL); } tm = profile_end(tm); ILOG("%d trials of gettimeofday: %s", trials, profile_msg(tm)); tm = profile_start(); for (int i = 0; i < trials; i++) { t += os_hrtime(); } tm = profile_end(tm); ILOG("%d trials of os_hrtime: %s", trials, profile_msg(tm)); tm = profile_start(); for (int i = 0; i < trials; i++) { t += os_utime(); } tm = profile_end(tm); ILOG("%d trials of os_utime: %s", trials, profile_msg(tm)); ILOG("%zu", t);
* Merge #10369 from janlazo/vim-8.0.1202Justin M. Keyes2019-06-29
|\ | | | | vim-patch:8.0.{1202,1259}
| * vim-patch:8.0.1259: search test can be flakyJan Edmund Lazo2019-06-28
| | | | | | | | | | | | | | | | Problem: Search test can be flaky. Solution: Use WaitFor() instead of a delay. Make it possible to pass a funcref to WaitFor() to avoid the need for global variables. (James McCoy, closes vim/vim#2282) https://github.com/vim/vim/commit/13deab8d08145c1f6e2a3e82cb547bc7f87a3686
| * vim-patch:8.0.1238: incremental search only shows one matchJan Edmund Lazo2019-06-28
| | | | | | | | | | | | | | Problem: Incremental search only shows one match. Solution: When 'incsearch' and and 'hlsearch' are both set highlight all matches. (haya14busa, closes vim/vim#2198) https://github.com/vim/vim/commit/2e51d9a0972080b087d566608472928d5b7b35d7
| * vim-patch:8.0.1202: :wall gives an errof for a terminal windowJan Edmund Lazo2019-06-28
|/ | | | | | | Problem: :wall gives an errof for a terminal window. (Marius Gedminas) Solution: Don't try writing a buffer that can't be written. (Yasuhiro Matsumoto, closes vim/vim#2190) https://github.com/vim/vim/commit/059db5c29ffef283a4b90bab9228708fa32e3dd2
* build: remove patch: luv-Add-missing-definitions-for-MinGW [skip travis] #10360Daniel Hahler2019-06-28
| | | | Added in 4ae7acd15 initially, but seems to not be necessary anymore, and/or should be send upstream otherwise.
* build: CMake: remove usage of USE_BUNDLED_X in main project (#10354)Daniel Hahler2019-06-28
| | | | | | | | | | | | | | | | | | The variables are not meant to be defined there really, but only with the third-party project. Using them, e.g. with the following, would actually result in libvterm not being found then: make CMAKE_EXTRA_FLAGS="-DUSE_BUNDLED_LIBVTERM=ON" \ DEPS_CMAKE_FLAGS="-DUSE_BUNDLED=OFF -DUSE_BUNDLED_LIBVTERM=ON" In https://github.com/neovim/neovim/pull/6357 they were renamed to `USE_BUNDLED_X` from `X_USE_BUNDLED`, but the above reasoning applies to the old names, too. Internally `CMAKE_PREFIX_PATH` is used to add the built/bundled third party packages for `find_package`, so there is no reason to e.g. query the values via `load_cache` for example from the third-party project.
* Merge #10349 from janlazo/vim-8.0.0935Justin M. Keyes2019-06-27
|\ | | | | vim-patch:8.0.{935,1013,1100,1119}
| * vim-patch:8.0.1120: :tm means :tmap instead of :tmenuJan Edmund Lazo2019-06-27
| | | | | | | | | | | | Problem: :tm means :tmap instead of :tmenu. (Taro Muraoka) Solution: Move the new entry below the old entry. (closes vim/vim#2102) https://github.com/vim/vim/commit/63c4e8a1986796094e6f15b893f2deccdf482617
| * vim-patch:8.0.1100: stuck in redraw loop when 'lazyredraw' is setJan Edmund Lazo2019-06-26
| | | | | | | | | | | | | | Problem: Stuck in redraw loop when 'lazyredraw' is set. Solution: Don't loop on update_screen() when not redrawing. (Yasuhiro Matsumoto, closes vim/vim#2082) https://github.com/vim/vim/commit/072412ed45aa20a67aaa7e387d7f6bf59c7d3fbe
| * vim-patch:8.0.1013: terminal window behaves different from a buffer with changesJan Edmund Lazo2019-06-26
| | | | | | | | | | | | | | | | Problem: A terminal window with a running job behaves different from a window containing a changed buffer. Solution: Do not set 'bufhidden' to "hide". Fix that a buffer where a terminal used to run is listed as "[Scratch]". https://github.com/vim/vim/commit/e561a7e2fa511d643c9692d26f4cf65378fd1983
| * vim-patch:8.0.0935: cannot recognize a terminal buffer in :ls outputJan Edmund Lazo2019-06-26
| | | | | | | | | | | | Problem: Cannot recognize a terminal buffer in :ls output. Solution: Use R for a running job and F for a finished job. https://github.com/vim/vim/commit/304b64c9e6957fa3f552e0540ca786139b39a1c4
* | CMakeLists: remove/cleanup passing of CMAKE_SYSTEM_NAME (#10351)Daniel Hahler2019-06-27
| | | | | | | | | | | | | | | | Just set it from `${CMAKE_HOST_SYSTEM_NAME}` directly, instead of passing it from the main CMake file (CMAKE_SYSTEM_NAME defaults to it, but is empty in script mode). Initially added in 9ce81f7b2, but then even used with unrelated commands (that do not use RunTests.cmake, e.g. 221f6fffa).
* | build: BuildLua: fix check for mingw [skip ci] (#10352)Daniel Hahler2019-06-27
| |
* | ci: AppVeyor: GCOV_ERROR_FILE: head/tail [skip ci] (#10335)Daniel Hahler2019-06-27
|/
* Makefile: fix regression with "make functionaltest-lua" (#10346)Daniel Hahler2019-06-26
| | | Regressed in 69eb4fa0c.