aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | vim-patch:8.1.2152: problems navigating tags file on MacOS CatalinaJan Edmund Lazo2019-10-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Problems navigating tags file on MacOS Catalina. Solution: Use fseek instead of lseek. (John Lamb, fixes vim/vim#5061) https://github.com/vim/vim/commit/27fc8cab227e30f649f52e74efd58ad56d21e9bb
* | | | | | test/functional: retry/Screen: failure instead of error #11173Daniel Hahler2019-10-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Running out of retries, or unexpected screen state should make the test FAIL, not ERROR. - Uses levels to report the location of the caller. - Improve message with retry-failure (formatting). Before: [ RUN ] test: 103.53 ms ERR test/functional/helpers.lua:388: retry() attempts: 1 test/functional/ui/screen.lua:587: Row 1 did not match. Expected: |*X^ | |{0:~ }| |{0:~ }| | | Actual: |*^ | |{0:~ }| |{0:~ }| | | To print the expect() call that would assert the current screen state, use screen:snapshot_util(). In case of non-deterministic failures, use screen:redraw_debug() to show all intermediate screen states. stack traceback: test/functional/helpers.lua:388: in function 'retry' test/functional/test_spec.lua:24: in function <test/functional/test_spec.lua:23> After: [ RUN ] test: 105.22 ms FAIL test/functional/test_spec.lua:24: stopping after 1 retry() attempts. test/functional/test_spec.lua:25: Row 1 did not match. Expected: |*X^ | |{0:~ }| |{0:~ }| | | Actual: |*^ | |{0:~ }| |{0:~ }| | | To print the expect() call that would assert the current screen state, use screen:snapshot_util(). In case of non-deterministic failures, use screen:redraw_debug() to show all intermediate screen states. stack traceback: test/functional/helpers.lua:389: in function 'retry' test/functional/test_spec.lua:24: in function <test/functional/test_spec.lua:23>
* | | | | | vim-patch:8.1.2180: Error E303 is not useful when 'directory' is empty (#11257)Daniel Hahler2019-10-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Error E303 is not useful when 'directory' is empty. Solution: Skip the error message. (Daniel Hahler, vim/vim#5067) https://github.com/vim/vim/commit/00e192becd50a38cb21a1bc3f86fcc7a21f8ee88
* | | | | | vim-patch:8.1.2182: test42 seen as binary by git diff #11256Daniel Hahler2019-10-19
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test42 seen as binary by git diff. Solution: Add .gitattributes file. Make explicit that 'cpo' does not contain 'S'. (Daniel Hahler, closes vim/vim#5072) https://github.com/vim/vim/commit/5b39d7adb0b9f02afe242f607d4c96250f06965d
* | | | | tests: let_spec: enable "multibyte env var to child process" (#11233)Daniel Hahler2019-10-18
| | | | |
* | | | | build: do not build test fixtures by default (#11230)Daniel Hahler2019-10-18
| | | | | | | | | | | | | | | | | | | | | | | | | - tty-test is also used on Windows - FUNCTIONALTEST_PREREQS: add printenv-test
* | | | | tests: fix non-controversial misuse of `pending` (#11247)Daniel Hahler2019-10-18
| | | | | | | | | | | | | | | Ref: https://github.com/neovim/neovim/pull/11184
* | | | | vim-patch:8.1.2140: "gk" and "gj" do not work correctly in number column #11208Jan Edmund Lazo2019-10-17
| | | | | | | | | | | | | | | | | | | | | | | | | Problem: "gk" and "gj" do not work correctly in number column. Solution: Allow for a negative "curswant". (Zach Wegner, closes vim/vim#4969) https://github.com/vim/vim/commit/ceba3dd5187788e09f65bd41b07b40f6f9aab953
* | | | | Merge #11215 from erw7/vim-8.1.0084Justin M. Keyes2019-10-17
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.1.{84,85,361}
| * | | | | vim-patch 8.1.0361: remote user not used for completionerw72019-10-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Remote user not used for completion. (Stucki) Solution: Use $USER too. (Dominique Pelle, closes #3407) https://github.com/vim/vim/commit/6b0b83f768cf536b34ce4d3f2de6bf62324229aa
| * | | | | vim-patch 8.1.0085: no test for completing user name and languageerw72019-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for completing user name and language. Solution: Add tests. (Dominique Pelle, closes #2978) https://github.com/vim/vim/commit/5f8f2d378a4f6d7db12806f3e35ec6f7fc6bd1f3
| * | | | | vim-patch 8.1.0084: user name completion does not work on MS-Windowserw72019-10-13
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Problem: User name completion does not work on MS-Windows. Solution: Use NetUserEnum() to get user names. (Yasuhiro Matsumoto) https://github.com/vim/vim/commit/828c3d70833a0689cc07581f2a67d06430675da5
* | | | | vim-patch:8.1.2096: too many #ifdefs #11229Daniel Hahler2019-10-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Too many #ifdefs. Solution: Graduate FEAT_COMMENTS. https://github.com/vim/vim/commit/8c96af9c05bfcac2d5ae081e098d4863db561511 Fixes https://github.com/vim/vim/issues/4972.
* | | | | scripts/vim-patch.sh: lazily update Vim source (#11207)Daniel Hahler2019-10-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gets done only initially, for `-l`, and when a commit cannot be found. Also provide more compact instructions with `-l` / `show_vimpatches`.
* | | | | tests: tui_spec: fix waiting for terminal to be ready (#11232)Daniel Hahler2019-10-15
| | | | | | | | | | | | | | | | | | | | | | | | | The screen would have '-- TERMINAL --' already initially. Related to flakiness of "TUI FocusGained/FocusLost in terminal-mode".
* | | | | deps: pass LDFLAGS+=-static (#11138)Daniel Hahler2019-10-14
| | | | | | | | | | | | | | | | | | | | | | | | | This is required when `build_old_libs=no` is used in `libtool`. Fixes https://github.com/neovim/neovim/issues/11198
* | | | | mac: fix "tags file not sorted" bug on Catalina (#11222)dm1try2019-10-14
| | | | | | | | | | | | | | | | | | | | | | | | | I/O in Catalina is currently known to be broken. This commit works around a pesky bug and also makes the code more consistent by removing the mix of C file and standard I/O. Fixes https://github.com/neovim/neovim/issues/11196
* | | | | Merge pull request #11206 from bfredl/testattrBjörn Linse2019-10-13
|\ \ \ \ \ | | | | | | | | | | | | tests/refactor: make screen.ui use "linegrid" representation internally
| * | | | | tests/ui: completely delete "attr_ignore" featureBjörn Linse2019-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All existing usages are ad-hoc/random/lazyness. Generating attribute specifications is not hard since four years, just do it always.
| * | | | | tests/ui: remove unnecessary screen:detach()Björn Linse2019-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is perfectly fine and expected to detach from the screen just by the UI disconnecting from nvim or exiting nvim. Just keep detach() in screen_basic_spec, to get some coverage of the detach method itself. This avoids hang on failure in many situations (though one could argue that detach() should be "fast", or at least "as fast as resize", which works in press-return already). Never use detach() just to change the size of the screen, try_resize() method exists for that specifically.
| * | | | | tests/ui: cleanup illegitimate usages of "attr_ignore"Björn Linse2019-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "attr_ignore" is an anti-pattern, with snapshot_util() just include all the highlights already.
| * | | | | tests/ui: make screen.lua use "linegrid" representation internallyBjörn Linse2019-10-13
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR #8221 took a short-cut when implementing the tests: screen.lua would translate the linegrid highlight ids back into the old per-cell attribute description. Apart from cleaning up technical debt, this enables to check both rgb and cterm colors in the same expect(), which previously was needlessly restricted to ext_hlstate tests only.
* / / / / scripts/vim-patch.sh -l: display commit subjectsDaniel Hahler2019-10-13
|/ / / / | | | | | | | | | | | | Closes https://github.com/neovim/neovim/pull/11182.
* | | | recovery mode (-r/-L): use headless_mode (#11187)Daniel Hahler2019-10-11
| | | | | | | | | | | | Fixes https://github.com/neovim/neovim/issues/11181.
* | | | vim-patch:8.1.2125: fnamemodify() fails when repeating :eJustin M. Keyes2019-10-10
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Fnamemodify() fails when repeating :e. Solution: Do not go before the tail. (Rob Pilling, closes vim/vim#5024) https://github.com/vim/vim/commit/b189295b72030f00c45c30d3daecf85d457221f8
* | | | fnamemodify: fix handling of :r after :e #11165Rob Pilling2019-10-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Test fnamemodify() - Test handling of `expand("%:e:e:r")`. - Fix :e:e:r on filenames with insufficiently many extensions During `fnamemodify()`, ensuring that we don't go before the filename's tail is insufficient in cases where we've already handled a ":e" modifier, for example: ``` "path/to/this.file.ext" :e:e:r:r ^ ^-------- *fnamep +------------- tail ``` This means for a ":r", we'll go before `*fnamep`, and outside the bounds of the filename. This is both incorrect and causes neovim to exit with an allocation error. We exit because we attempt to calculate `s - *fnamep` (line 23948). Since `s` is before `*fnamep`, we caluclate a negative length, which ends up being interpreted as an amount to allocate, causing neovim to exit with ENOMEM (`memory.c:xmalloc`). We must instead ensure we don't go before `*fnamep` nor `tail`. The check for `tail` is still relevant, for example: ``` "path/to/this.file.ext" :r:r:r ^ ^------------- tail +--------------------- *fnamep ``` Here we don't want to go before `tail`. close #11165
* | | | test: "!:&" works with powershell #11201Jan Edmund Lazo2019-10-10
| | | | | | | | | | | | Removed 'echo' alias because it does not behave like POSIX echo.
* | | | Remove "highbright bold" conversion. Fixes #11190Björn Linse2019-10-10
|/ / / | | | | | | | | | | | | | | | When using TUI host terminal should take care of this (regardless if 'termguicolors' is active or not). For GUI the behavior doesn't make sense (GUI should display bold attr as bold always).
* | | update_version_stamp: redirect stderr on first try, --first-parent #11186Daniel Hahler2019-10-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid noise during builds: > fatal: No annotated tags can describe '417449f468c4ba186954f6295b3338fb55ee7b4a'. > However, there were unannotated tags: try --tags. This might be useful in general, but is expected to not happen - and falling back is OK then. The fallback command would still display errors then. It also uses `--first-parent`, which is important for when a release branch gets merged back.
* | | doc: update shellquote for powershell #11122Jan Edmund Lazo2019-10-10
| | | | | | | | | shellquote is not treated like shellxquote for non-quote values.
* | | third-party: upgrade libvterm to 0.1.2 (#11177)Daniel Hahler2019-10-09
| | |
* | | ci: AppVeyor: coverage for Lua (Windows) (#10426)Daniel Hahler2019-10-09
| | |
* | | ci: OpenBSD: enable functionaltest (#11178)Daniel Hahler2019-10-08
| | |
* | | Merge #11077 'vim-patch:8.1.{1354,1356,1362,1588}'Justin M. Keyes2019-10-07
|\ \ \
| * | | vim-patch:8.1.1588: in :let-heredoc line continuation is recognizedJurica Bradaric2019-10-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: In :let-heredoc line continuation is recognized. Solution: Do not consume line continuation. (Ozaki Kiichi, closes vim/vim#4580) https://github.com/vim/vim/commit/e96a2498f9a2d3e93ac07431f6d4afd77f30afdf
| * | | vim-patch:8.1.1362: code and data in tests can be hard to readJurica Bradaric2019-10-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Code and data in tests can be hard to read. Solution: Use the new heredoc style. (Yegappan Lakshmanan, closes vim/vim#4400) https://github.com/vim/vim/commit/c79745a82faeb5a6058e915ca49a4c69fa60ea01
| * | | vim-patch:8.1.1356: some text in heredoc assignment ends the textJurica Bradaric2019-10-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Some text in heredoc assignment ends the text. (Ozaki Kiichi) Solution: Recognize "let v =<<" and skip until the end. https://github.com/vim/vim/commit/8471e57026714c5a0faf89288ceef5231fb88d4f
| * | | vim-patch:8.1.1354: getting a list of text lines is clumsyJurica Bradaric2019-10-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Getting a list of text lines is clumsy. Solution: Add the =<< assignment. (Yegappan Lakshmanan, closes vim/vim#4386) https://github.com/vim/vim/commit/f5842c5a533346c4ff41ff666e465c85f1de35d5
* | | | cmake: only set LUA_PRG with successful check (#11172)Daniel Hahler2019-10-07
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is relevant for when using `USE_BUNDLED_LUAJIT=ON` with `USE_BUNDLED_LUAROCKS=OFF`, and then building without the necessary modules being installed/activated there yet: it would check the other (system) "lua" interpreters also, and in case all failed keep the `LUA_PRG` in the cache for the last failed entry - making it not re-check the previous ones on the next build (after you might have activated your custom LuaRocks installation). Only setting LUA_PRG if the check was successful handles the case better where it is configured already - we should not try to re-configure it then.
* | | Merge pull request #11154 from blueyed/bump-tree-sitterDaniel Hahler2019-10-07
|\ \ \ | | | | | | | | ci: upgrade tree-sitter from 0.15.2 to 0.15.9
| * | | lintDaniel Hahler2019-10-06
| | | |
| * | | ci: upgrade tree-sitter from 0.15.2 to 0.15.9Daniel Hahler2019-10-06
| | | | | | | | | | | | | | | | tree-sitter-c is still at 0.15.2 though.
* | | | ci: use cluacov for better performance (#11152)Daniel Hahler2019-10-07
| | | |
* | | | Merge #11170 from janlazo/vim-8.1.2120Justin M. Keyes2019-10-06
|\ \ \ \ | | | | | | | | | | vim-patch:8.1.2120
| * | | | Remove dead codeJan Edmund Lazo2019-10-06
| | | | |
| * | | | vim-patch:8.1.2120: some MB_ macros are more complicated than necessaryJan Edmund Lazo2019-10-06
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some MB_ macros are more complicated than necessary. (Dominique Pelle) Solution: Simplify the macros. Expand inline. https://github.com/vim/vim/commit/1614a14901558ca091329315d14a7d5e1b53aa47
* | | | ci: submit_coverage: run luacov actually (#11169)Daniel Hahler2019-10-07
| | | | | | | | | | | | Apparently this got lost with #11127 / 77a551b65.
* | | | test/old: add test_fnamemodify.vim (#11168)Daniel Hahler2019-10-06
| | | | | | | | | | | | | | | | | | | | Moved to a new-style test in vim/vim@610cc1b9b (v7.4.1652). Ref: https://github.com/neovim/neovim/pull/11165#issuecomment-538785588
* | | | tests: retry: "wait() evaluates the condition on given interval" (#11155)Daniel Hahler2019-10-06
|/ / / | | | | | | Ref: https://github.com/neovim/neovim/issues/11137
* | | tests/functional: keep $TMPDIR in env (#11163)Daniel Hahler2019-10-06
| | |