| Commit message (Collapse) | Author | Age |
... | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Changing an autoload name into a script file name is inefficient.
Solution: Remember the last replaced #. (Ozaki Kiichi, closes vim/vim#4618)
https://github.com/vim/vim/commit/28fc247f8d94a1cfabbcf2691ca942bde96f6d2f
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Terminal test fails when term_getansicolors() is missing.
Diff test fails without +rightleft. (Dominique Pelle)
Solution: Check if term_getansicolors() is supported. (closes vim/vim#4597)
https://github.com/vim/vim/commit/981d9dc8f3d7a470eb3be7ba60fe1c1f31e7f470
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Build with EXITFREE but without +arabic fails.
Solution: Rename the function and adjust #ifdefs. (closes vim/vim#4613)
https://github.com/vim/vim/commit/48ac671fe5cb5a7c2d5263d2f122e5e903022e30
|
| |/
| |
| |
| |
| |
| | |
Problem: 'numberwidth' can only go up to 10.
Solution: Allow up to 20. (Charlie Stanton, closes vim/vim#4584)
https://github.com/vim/vim/commit/f8a071265535b8cc43e50a81f4d5049883ca50e4
|
|/
|
| |
Follow-up to https://github.com/neovim/neovim/pull/10428#discussion_r300813498
|
|
|
|
|
| |
Problem: Crash when dumping profiling data.
Solution: Reset flag indicating that initialization was done.
https://github.com/vim/vim/commit/79c2ad50b8069bbb3281bcabe412bd6641756724
|
| |
|
|\
| |
| | |
screen: disable redrawing inside VimResized
|
| |
| |
| |
| |
| |
| | |
Note: test doesn't fail on master. I cannot reproduce the glitches with
-u NONE, probably it requires interfering events. But add some coverage
for these checks at least.
|
|\ \
| |/
|/| |
highlight: show "hi Group" message correctly when not using the screen
|
|/
|
|
|
|
|
| |
ext_message doesn't set msg_col. Add a space and let client deal with
wrapping. When using silent redirect show the unwrapped message form.
Removed check is already part of msg_advance()
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* build: FindLibIntl: fix warning about CMP0075
The common pattern elsewhere to set this only during the check, and here
it was not unset, resulting in a warning later (on Alpine 3.10):
-- Found Iconv
-- Looking for pthread.h
CMake Warning (dev) at /usr/share/cmake/Modules/CheckIncludeFile.cmake:80 (message):
Policy CMP0075 is not set: Include file check macros honor
CMAKE_REQUIRED_LIBRARIES. Run "cmake --help-policy CMP0075" for policy
details. Use the cmake_policy command to set the policy and suppress this
warning.
CMAKE_REQUIRED_LIBRARIES is set to:
/usr/lib/libintl.so
For compatibility with CMake 3.11 and below this check is ignoring it.
Call Stack (most recent call first):
/usr/share/cmake/Modules/FindThreads.cmake:105 (CHECK_INCLUDE_FILE)
CMakeLists.txt:482 (find_package)
This warning is for project developers. Use -Wno-dev to suppress it.
-- Looking for pthread.h - found
* build: remove lists / REMOVE_ITEM around check_c_source_compiles
|
|\
| |
| | |
BuildLuarocks.cmake: use ROCKS_DIR
|
| |
| |
| |
| |
| |
| |
| | |
I found it often irritation to see the same comment about "DEPENDS"
there all the time. This uses the LuaRocks rock name as a comment
instead (which is hard to parse from the recipe), moves the note about
DEPENDS to the top (extended), and fixes/aligns some whitespace.
|
|/
|
|
|
|
|
|
| |
With LuaRocks 3 it will include the version, and this will make the
upgrade easier then.
It also makes sense to factor this out in general for clarity already.
Pulled out of https://github.com/neovim/neovim/pull/10292.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Flaky test (osx):
[ FAILED ] ...is/build/neovim/neovim/test/functional/lua/loop_spec.lua @ 23: vim.loop timer
...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:56: Expected objects to be the same.
Passed in:
(number) 0
Expected:
(number) 2
stack traceback:
...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:56: in function <...is/build/neovim/neovim/test/functional/lua/loop_spec.lua:23>
It was bumped from sleeping for 20ms to 50ms in d33aaa0f5f already.
|
|
|
|
|
|
| |
"auto" does not work, since Ninja does not appear as a TTY to gcc/clang,
but handles stripping escape codes itself when output is not to a TTY.
Ref: https://github.com/ninja-build/ninja/blob/ca041d88f4d6/src/build.cc#L158-L169
|
| |
|
|\
| |
| | |
Improve luacheck integration, fix more issues.
|
| |
| |
| |
| |
| |
| |
| |
| | |
- Move .luacheckrc to root, add read_globals=vim
- Simplify lualint target, run it on all lua files
- Lint preload.lua, but ignore W211
- Remove testlint target, included in lualint (and lint)
- Clean up .luacheckrc
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Fixes warning with:
> cmake -S third-party -B .deps -DUSE_BUNDLED=OFF -DUSE_BUNDLED_UNIBILIUM=0 -DUSE_BUNDLED_LIBTERMKEY=1
> The dependency target "unibilium" of target "libtermkey" does not exist.
|
|/
|
|
|
|
|
|
|
|
|
| |
Fix issue that increment expression is executable and pointer ext
pointing out of the buffer, if the pointer ext points to the terminating
NUL.
* Change termination condition judgment to one place
* Change first condition judgment
Change to not evaluate *(ext -1) in the first condition judgment.
* Change to use copy_option_part instead of STRLCPY
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is required to (re)build e.g. libluv when the version changes
(which triggers a new download).
With `make deps`, changing the `LUV_URL`/`LUV_SHA256`, and `make deps` again:
Before:
> Up-to-date: /home/daniel/Vcs/neovim/.deps/usr/lib/libluv.a
After:
> Installing: /home/daniel/Vcs/neovim/.deps/usr/lib/libluv.a
See with https://github.com/neovim/neovim/pull/10358 - where .deps
contained libluv 1.29, the merge updates it to 1.30, but then it failed
to link because `libluv.a` is considered to be up-to-date (after
downloading the new version).
Note that header files get installed, since they have the original time
stamp, but `libluv.a` is being generated (does not use the timestamp
from the archive here, but needs to get rebuild).
It could be argued that the build system of the included project should
catch/handle this, but it seems to be good practice to clean the binary
/ build dir with a new download to start from scratch.
Ref: https://gitlab.kitware.com/cmake/cmake/issues/19452
Also fixes cmake/BuildLuv / luv-static: use name with -DTARGET for
download command, and pass (shared) `SRC_DIR` explicitly instead.
|
|
|
|
| |
Use it to improve ls_spec: it should not use the user's real shell for
performance and other reasons.
|
|
|
|
| |
It was moved to become pending in 18127f64c, but the assertion should be
kept.
|
|
|
|
| |
- Ensure the opposite of the expected bg is active.
- Improves performance: 1.2s instead of 4.5s.
|
|
|
|
|
| |
This makes it easier to inspect the test failure, without having to open
two folds (typically), and scrolling around.
|
|
|
| |
In order that `$(DEPS_BUILD_DIR)/Makefile` exists under parallel make
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
| |
`retry()` would only try it three times (waiting for 20ms in between),
despite the large timeout.
Fixes https://github.com/neovim/neovim/issues/10265.
|
|\
| |
| | |
cmdline: remove local variables i and j from command_line_state
|
|/ |
|
|
|
| |
ref #6289
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* 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.
|
|\
| |
| | |
api/lua: make nvim_execute_lua use native lua floats, not special tables
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| | |
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
|
|\ \
| |/
|/| |
cmdline: correct the column position of wildoptions=pum popupmenu
|
|/
|
|
|
| |
- position might get invalid with "longest" match
- position might be wrong when completed pattern ends in "/"
|
|
|
| |
Since 0364f51 it would display a warning.
|
|
|
|
|
|
|
| |
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)).
|
|\
| |
| | |
luv callbacks: throw error on deferred methods instead of crashing
|
| |
| |
| |
| |
| | |
This avoids initializing libluv a second time if a plugin invokes
require'luv'. It is probably not an issue, but better to be safe.
|
| |
| |
| |
| |
| | |
Disable the use of deferred API functions in a fast lua callback
Correctly display error messages from a fast lua callback
|
| | |
|
| |
| |
| | |
Introduce os_file_settime(), remove cruft.
|
|/
|
|
| |
closes #10267
|