aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * vim-patch:8.1.0907: CI tests on AppVeyor are failingJan Edmund Lazo2019-07-25
| | | | | | | | | | | | Problem: CI tests on AppVeyor are failing. Solution: Reduce the recursiveness limit for regexp. https://github.com/vim/vim/commit/5382f12c910b7f8e46acdde5488f26a86f9fcac1
| * vim-patch:8.1.0905: complicated regexp causes a crashJan Edmund Lazo2019-07-25
| | | | | | | | | | | | Problem: Complicated regexp causes a crash. (Kuang-che Wu) Solution: Limit the recursiveness of addstate(). (closes vim/vim#3941) https://github.com/vim/vim/commit/5567ad48b66dff13670af52a48509059acc34dfe
| * regexp: use fixed types to avoid overflowJan Edmund Lazo2019-07-24
| |
| * Checks for overflow when parsing string to intGabriel2019-07-24
| |
| * vim-patch:8.1.0908: can't handle large value for %{nr}v in regexpJan Edmund Lazo2019-07-24
| | | | | | | | | | | | Problem: Can't handle large value for %{nr}v in regexp. (Kuang-che Wu) Solution: Give an error if the value is too large. (closes vim/vim#3948) https://github.com/vim/vim/commit/9403a2168db82b7de80f792984084bb3f00e2263
| * vim-patch:8.1.1746: ":dl" is seen as ":dlist" instead of ":delete"Jan Edmund Lazo2019-07-24
| | | | | | | | | | | | Problem: ":dl" is seen as ":dlist" instead of ":delete". Solution: Do not use cmdidxs2[] if the length is 1. (closes vim/vim#4721) https://github.com/vim/vim/commit/94f82cbacf76767b5ac32f813e1d670501dbd0e6
| * vim-patch:8.1.0903: struct uses more bytes than neededJan Edmund Lazo2019-07-24
| | | | | | | | | | | | Problem: Struct uses more bytes than needed. Solution: Reorder members of regitem_S. (Dominique Pelle, closes vim/vim#3936) https://github.com/vim/vim/commit/beb7574d6b0eea1cae70aa2913a690da56de5307
| * vim-patch:8.1.0899: no need to check restricted mode for setwinvar()Jan Edmund Lazo2019-07-24
|/ | | | | | Problem: No need to check restricted mode for setwinvar(). Solution: Remove check_restricted(). https://github.com/vim/vim/commit/e0fb7d1e38e1cf699412e212cda863420f5f8bd2
* coverage: use "cd" with gcovr (#10594)Daniel Hahler2019-07-25
| | | | | | This makes the invocation compatible for the upcoming gcovr 4.2 release, and is the correct way of invoking it. Ref: https://github.com/gcovr/gcovr/commit/a782972#commitcomment-34420728
* vim-patch:8.1.1740: exepath() doesn't work for "bin/cat" (#10556)Daniel Hahler2019-07-24
| | | | | | | | Problem: Exepath() doesn't work for "bin/cat". Solution: Check for any path separator. (Daniel Hahler, closes vim/vim#4724, closes vim/vim#4710) https://github.com/vim/vim/commit/d08b8c4c04db9433340df38d21f0e26878f28421 Fixes https://github.com/neovim/neovim/issues/10554.
* vim-patch:8.1.1738: testing lambda with timer is slow (#10590)Daniel Hahler2019-07-24
| | | | | | Problem: Testing lambda with timer is slow. Solution: Do not test timer accuracy, only that it works. (Daniel Hahler, closes vim/vim#4723) https://github.com/vim/vim/commit/9bc4dde45d45df732953491d0f2c3fd3b10a627e
* third-party: download: retry (#10599)Daniel Hahler2019-07-24
| | | | | | | | | | | | | | | | | | | | | | This is meant to handle the common case of failing to download libtermkey: FAILED: cd /home/travis/build/neovim/neovim/deps-downloads/libtermkey && /usr/local/cmake-3.12.4/bin/cmake -DPREFIX=/home/travis/nvim-deps/build -DDOWNLOAD_DIR=/home/travis/build/neovim/neovim/deps-downloads/libtermkey -DURL=http://www.leonerd.org.uk/code/libtermkey/libtermkey-0.21.1.tar.gz -DEXPECTED_SHA256=cecbf737f35d18f433c8d7864f63c0f878af41f8bd0255a3ebb16010dc044d5f -DTARGET=libtermkey -DUSE_EXISTING_SRC_DIR=OFF -P /home/travis/build/neovim/neovim/third-party/cmake/DownloadAndExtractFile.cmake && /usr/local/cmake-3.12.4/bin/cmake -E touch /home/travis/nvim-deps/build/src/libtermkey-stamp/libtermkey-download -- file: /home/travis/build/neovim/neovim/deps-downloads/libtermkey/libtermkey-0.21.1.tar.gz -- downloading... src='http://www.leonerd.org.uk/code/libtermkey/libtermkey-0.21.1.tar.gz' dst='/home/travis/build/neovim/neovim/deps-downloads/libtermkey/libtermkey-0.21.1.tar.gz' timeout='none' CMake Error at /home/travis/build/neovim/neovim/third-party/cmake/DownloadAndExtractFile.cmake:77 (message): error: downloading 'http://www.leonerd.org.uk/code/libtermkey/libtermkey-0.21.1.tar.gz' failed status_code: 6 status_string: "Couldn't resolve host name" log: Curl_ipv4_resolve_r failed for www.leonerd.org.uk Couldn't resolve host 'www.leonerd.org.uk' Closing connection 0 Co-Authored-By: Justin M. Keyes <justinkz@gmail.com>
* ci: RunTests: ensure that the logfile gets displayed (#10597)Daniel Hahler2019-07-24
| | | | | | This is meant to help with test failures where e.g. `busted` itself might crash or get killed. Ref: https://github.com/neovim/neovim/issues/10182#issuecomment-514450069
* sign: REMOVE FEAT_SIGN_ICONS, dead code #10595Jan Edmund Lazo2019-07-24
|
* tests: AppVeyor: fix test/functional/ex_cmds/arg_spec.lua (#10598)Daniel Hahler2019-07-24
| | | | | | | | | | | | | | | | | | | | For unknown reasons it does not have the trailing space in `:args` output there anymore: [ FAILED ] test/functional\ex_cmds\arg_spec.lua @ 11: :argument does not restart :terminal buffer test/functional\ex_cmds\arg_spec.lua:25: Expected objects to be the same. Passed in: (string) ' [term://.//4552:C:\Windows\system32\cmd.exe]' Expected: (string) ' [term://.//4552:C:\Windows\system32\cmd.exe] ' stack traceback: test/functional\ex_cmds\arg_spec.lua:25: in function <test/functional\ex_cmds\arg_spec.lua:11> The test is not about that though, and this can be made less strict by using `trim()`. The new test in `test_arglist.vim` for no trailing newline is OK, and contains trailing spaces. So this is likely due to the length of it exceeding the column width already.
* vim-patch:8.1.1737: :args command that outputs one line gives more promptDaniel Hahler2019-07-24
| | | | | | | | Problem: :args command that outputs one line gives more prompt. Solution: Only output line break if needed. (Daniel Hahler, closes vim/vim#4715) https://github.com/vim/vim/commit/949f1989cba8bf7653316c2b1444c26f1536bfab Closes https://github.com/neovim/neovim/pull/10147.
* vim-patch:8.0.1738: ":args" output is hard to readDaniel Hahler2019-07-24
| | | | | | | | | | | | | | | Problem: ":args" output is hard to read. Solution: Make columns with the names if the output is more than one line. https://github.com/vim/vim/commit/5d69da462f584a3aefb3427b127334bf9af3a4b0 vim-patch:8.0.1740: warning for signed-unsigned incompatibility Problem: Warning for signed-unsigned incompatibility. Solution: Change type from "char *" to "char_u *". (John Marriott) https://github.com/vim/vim/commit/405dadb63ea2b7aa4c8c659807506a35a8a9504c Removes ported legacy test that was re-added later. Ref: https://github.com/neovim/neovim/pull/10147#issuecomment-512609513
* tests: fix/improve Screen:expect_unchanged (#10577)Daniel Hahler2019-07-24
| | | | | | Do not sleep before collecting initial state. Ref: https://github.com/neovim/neovim/pull/10550#issuecomment-513670205
* ci: restore accidentally removed config (#10592)Daniel Hahler2019-07-24
|
* tests: re-enable "tab drag in tabline to the left moves tab left" (#10588)Daniel Hahler2019-07-24
| | | Ref: https://github.com/neovim/neovim/pull/4874
* Merge #10558 from erw7/fix-text-overrides-line-number2Justin M. Keyes2019-07-23
|\ | | | | Windows: Fix text overrides line number
| * env: invalid pointer after os_setenv() #10558erw72019-07-23
| |
| * win/TUI: workaround libuv LF => CRLF conversion #10558erw72019-07-23
|/ | | | | | Replace cud1 with \x1b[B because \n is CRLF on Windows (due to libuv). fix #9461
* Revert "vim-patch:8.0.1723: using one item array size declaration is ↵Jan Edmund Lazo2019-07-23
| | | | | misleading" (#10583) This reverts commit 315076a26d225a3bdc34a2cbba1f124adf4e1087.
* build: GetBinaryDeps: move include, fix doc (#10579)Daniel Hahler2019-07-22
|
* shell-test: fix REP for count larger than uint8_t (#10581)Daniel Hahler2019-07-22
|
* PVS/V1019: "readability" warning #10566Ihor Antonov2019-07-22
|
* tests: make TERM=interix test pending (#10576)Daniel Hahler2019-07-22
| | | | Ref: https://github.com/neovim/neovim/issues/10179 Ref: https://github.com/neovim/neovim/pull/9494
* build: fix gcc warnings #10568ddcien2019-07-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gcc version 8.3.0 (Debian 8.3.0-6) ../src/nvim/ex_docmd.c: In function ‘commands_array’: ../src/nvim/ex_docmd.c:10192:36: warning: ‘%ld’ directive output may be truncated writing between 1 and 19 bytes into a region of size 10 [-Wformat-truncation=] snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~ ../src/nvim/ex_docmd.c:10192:37: note: format string is defined here snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ../src/nvim/ex_docmd.c:10192:36: note: directive argument in the range [0, 9223372036854775807] snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~ ../src/nvim/ex_docmd.c:10192:9: note: ‘snprintf’ output between 2 and 20 bytes into a destination of size 10 snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../src/nvim/ex_docmd.c:10205:36: warning: ‘%ld’ directive output may be truncated writing between 1 and 19 bytes into a region of size 10 [-Wformat-truncation=] snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~ ../src/nvim/ex_docmd.c:10205:37: note: format string is defined here snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ../src/nvim/ex_docmd.c:10205:36: note: directive argument in the range [0, 9223372036854775807] snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~ ../src/nvim/ex_docmd.c:10205:9: note: ‘snprintf’ output between 2 and 20 bytes into a destination of size 10 snprintf(str, sizeof(str), "%" PRId64, (int64_t)cmd->uc_def); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../src/nvim/spell.c: In function ‘did_set_spelllang’: ../src/nvim/spell.c:1631:61: warning: ‘%s’ directive output may be truncated writing up to 254 bytes into a region of size 84 [-Wformat-truncation=] "autocmd VimEnter * call spellfile#LoadFile('%s')|set spell", ^~ ../src/nvim/spell.c:2063:25: spell_load_lang(lang); ~~~~ ../src/nvim/spell.c:1630:7: note: ‘snprintf’ output between 57 and 311 bytes into a destination of size 128 snprintf(autocmd_buf, sizeof(autocmd_buf), ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ "autocmd VimEnter * call spellfile#LoadFile('%s')|set spell", ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ lang); ~~~~~ ../src/nvim/window.c: In function ‘win_rotate’: ../src/nvim/window.c:1780:22: warning: ‘wp2’ may be used uninitialized in this function [-Wmaybe-uninitialized] wp2->w_pos_changed = true; ^ ../src/nvim/window.c:1779:22: warning: ‘wp1’ may be used uninitialized in this function [-Wmaybe-uninitialized] wp1->w_pos_changed = true;
* vim-patch:8.1.0706: introduce :redrawtabline #10570Jan Edmund Lazo2019-07-22
| | | | | | | Problem: Tabline is not always redrawn when something that is used in 'tabline' changes. Solution: Add ":redrawtabline" so that a plugin can at least cause the redraw when needed. https://github.com/vim/vim/commit/e12bab3144af8943937bd0ff4bc57f04e53037b3
* build: remove -Wno-array-bounds workaround #10484Daniel Hahler2019-07-22
| | | | | | | | Might not be necessary anymore, at least it is fine on gcc 9.1, and since the bug is still open/unresolved, something else might have fixed it. Ref: 4b81f627c Ref: #7083
* test/helpers: improve pattern with module functions (#10421)Daniel Hahler2019-07-22
| | | | | | Benefits: - less lines, especially less results when grepping - makes it clearer what is exported
* vim-patch:8.1.1724: too much overhead checking for CTRL-C #10564Jan Edmund Lazo2019-07-21
| | | | | | Problem: Too much overhead checking for CTRL-C while processing text. Solution: Increase BREAKCHECK_SKIP. Remove the difference for when built with the GUI. (suggested by Andy Massimino, closes vim/vim#4708) https://github.com/vim/vim/commit/b4fe0eb4b4fe52a68a1df05162c03fe51c2fce79
* Merge #10561 from justinmk/os_can_exeJustin M. Keyes2019-07-21
|\ | | | | jobstart(), system(): use $PATHEXT-resolved exe
| * os_can_exe: remove char_uJustin M. Keyes2019-07-21
| |
| * win: jobstart(), system(): $PATHEXT-resolve exeJustin M. Keyes2019-07-21
|/ | | | | | | | | | | | Windows: In order for jobstart(['foo']), system(['foo']) to find "foo.cmd", we must replace "foo" with "foo.cmd" before sending `argv` to process_spawn(). Rationale: jobstart([…]), system([…]) "executable" semantics should be consistent with the VimL executable() function. fix #9569 related: #10554
* Merge #10555 from janlazo/vim-8.1.1720Justin M. Keyes2019-07-21
|\ | | | | vim-patch:8.1.{856,1720}
| * move: assert nonnull wp pointerJan Edmund Lazo2019-07-20
| |
| * vim-patch:8.1.0856: when scrolling a window the cursorline is not always updatedJan Edmund Lazo2019-07-20
| | | | | | | | | | | | | | | | | | Problem: When scrolling a window other than the current one the cursorline highlighting is not always updated. (Jason Franklin) Solution: Call redraw_for_cursorline() after scrolling. Only set w_last_cursorline when drawing the cursor line. Reset the lines to be redrawn also when redrawing the whole window. https://github.com/vim/vim/commit/bbb5f8d4c2cbc5f48556008875f57cbe7fc4ac6c
| * vim-patch:8.1.1720: crash with very long %[] patternJan Edmund Lazo2019-07-20
|/ | | | | | Problem: Crash with very long %[] pattern. (Reza Mirzazade farkhani) Solution: Check for reg_toolong. (closes vim/vim#4703) https://github.com/vim/vim/commit/2a5b52758bb327b89d22660cc28c157ec29782e5
* vim-patch:8.1.0789: session sets v:errmsg #10553Jan Edmund Lazo2019-07-20
| | | | | Problem: Sourcing a session sets v:errmsg. Solution: Use "%argdel" instead of "argdel *". (Jason Franklin) https://github.com/vim/vim/commit/555de4e3b2881b0d9a72242ecc2ba26b5c698c85
* test: shell-test.c: "REP" command: flush, wait 1ms #10552Justin M. Keyes2019-07-20
| | | | | | Typically most shell output is the result of non-trivial work, so it would not blast stdout instantaneously. To more closely simulate that typical scenario, change `shell-test REP` to wait 1 millisecond between iterations.
* Merge #10550 'screen.lua: expect_unchanged()'Justin M. Keyes2019-07-20
|\
| * screen.lua: always print keyword-args snapshotJustin M. Keyes2019-07-20
| |
| * screen.lua: expect_unchanged(), get_snapshot()Justin M. Keyes2019-07-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Factor `get_snapshot()` out of `print_snapshot()`, so that callers can get a table (for use with `expect()`) instead of the string form. Try to use this to fix indeterminism in `searchhl_spec.lua`. - Since the screen state is collected by `screen:expect_unchanged()`, we don't need a deterministic initial state (which would then be hardcoded into the test). This allows us to check "did anything change in the last N ms?" rather than "did anything change compared to a hardcoded screen-state?" - This may end up fruitless, because `expect_unchanged()` depends on timing to wait for an initial "current state".
* | vim-patch:8.1.0754: preferred column when setting 'cursorcolumn' #10549Jan Edmund Lazo2019-07-20
|/ | | | | | Problem: Preferred column is lost when setting 'cursorcolumn'. Solution: Change option flag to P_RWINONLY. (Takayuki Kurosawa, closes vim/vim#3806) https://github.com/vim/vim/commit/177ab9e0262b1b3a6120bea655864ead487210e5
* test: shell-test.c: flush stdout for REP #10548Justin M. Keyes2019-07-20
| | | fix #10534
* [RFC]vim-patch:8.1.{749,1715} #10545Jan Edmund Lazo2019-07-20
| | | | | | | | | | | | | | | * vim-patch:8.1.1715: emoji characters are seen as word characters for spelling Problem: Emoji characters are seen as word characters for spelling. (Gautam Iyer) Solution: Exclude class 3 from word characters. https://github.com/vim/vim/commit/06e6377009c5763639310fa3bf892dec27a63334 * vim-patch:8.1.0749: error message contains garbage Problem: Error message contains garbage. (Dominique Pelle) Solution: Use correct pointer to failed expression. https://github.com/vim/vim/commit/6acc79f5d4b9d5b02f4ab21ec885e68acc13a2e2
* reltimefloat(): allow negative result #10544Justin M. Keyes2019-07-20
| | | | | | | | | For "backwards" duration, reltimefloat() should return negative value like its counterpart reltimestr(). ref bab24a88ab48 ref 06af88cd72ea ref #10521 fix #10452
* doc [ci skip] #10383Justin M. Keyes2019-07-19
| | | | - test/README.md: document luassert `TableFormatLevel` - CONTRIBUTING.md: absorb parts of the old "Development tips" wiki page