aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #8746 from oni-link/fix.rpcstart2Björn Linse2018-07-15
|\ | | | | Fix crash in channel_job_start() in combination with rpc and unknown command
| * channel.c: Prevent channel_destroy_early() from freeing uninitialized rpc stuffoni-link2018-07-15
|/ | | | | | | | Without this patch the following commands would crash: :call rpcstart('unknown_command') :set sh=unknown_command :call jobstart('ls',{'rpc':1})
* Merge pull request #8737 from dimbleby/overly-general-waitpidJames McCoy2018-07-13
|\ | | | | Only waitpid() for processes that we care about
| * Only waitpid() for processes that we care aboutDavid Hotham2018-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems as though in an AppImage there's an extra child process that dies at some early point, before we have set up a SIGCHLD handler. So when we later get a SIGCHLD from a child that we do care about, waitpid(-1, ...) tells us about the extra child - and we don't notice that the interesting child has exited. Or something like that! See also: * https://patchwork.kernel.org/patch/9949491/ in which perf hit something similar * discussion at the AppImage repository: https://github.com/AppImage/AppImageKit/issues/812#issuecomment-404662110. Fix is to be explicit about which process we are waitpid()'ing for, so we never need be distracted by children that we don't know about. Fixes #8104
* | vim-patch:8.0.0630: it is not easy to work on lines without a match (#8734)Jan Edmund Lazo2018-07-13
| | | | | | | | | | | | | | Problem: The :global command does not work recursively, which makes it difficult to execute a command on a line where one pattern matches and another does not match. (Miles Cranmer) Solution: Allow for recursion if it is for only one line. (closes vim/vim#1760) https://github.com/vim/vim/commit/f84b122a99da75741ae686fabb6f81b8b4755998
* | Merge #8736 'vim-patch:8.0.0697: stable keymap.h'Justin M. Keyes2018-07-13
|\ \ | |/ |/|
| * keymap: add commented events to match 8.0.0697Jan Edmund Lazo2018-07-13
| | | | | | | | KE_MOUSEMOVE, KE_CANCEL are added in later patches.
| * tests: <SNR> is represented as 'R' (ASCII)Jan Edmund Lazo2018-07-12
| |
| * vim-patch:8.0.0697: recorded key sequences may become invalidJan Edmund Lazo2018-07-10
| | | | | | | | | | | | | | Problem: Recorded key sequences may become invalid. Solution: Add back KE_SNIFF removed in 7.4.1433. Use fixed numbers for the key_extra enum. https://github.com/vim/vim/commit/8858498516108432453526f07783f14c9196e112
* | terminal: handle &confirm and :confirm on unloading (#8726)Marco Hinz2018-07-12
| | | | | | | | | | | | Show a proper confirmation dialog when trying to unload a terminal buffer while the confirm option is set or when :confirm is used. Fixes https://github.com/neovim/neovim/issues/4651
* | Merge pull request #8721 from bfredl/quickresizeBjörn Linse2018-07-12
|\ \ | | | | | | Fix redrawing issues with narrow screen and remove extra wait in resize tests
| * | screen: truncate showmode messagesBjörn Linse2018-07-12
| | | | | | | | | | | | | | | Before this, bottom of screen got messed up when modemsg (+ one extra space to not force terminal scroll) didn't fit on one line.
| * | tests/screen.lua: treat "resize" like any other eventBjörn Linse2018-07-11
| | | | | | | | | | | | Saves ~10 seconds in UI tests
* | | vim-patch:8.0.0{474,475,492,633,1251} (#8725)Jan Edmund Lazo2018-07-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * vim-patch:8.0.0474: the client-server feature is not tested Problem: The client-server feature is not tested. Solution: Add a test. https://github.com/vim/vim/commit/15bf76d40be1f1622ff5cc16596c308e76e2ca94 * vim-patch:8.0.0475: not enough testing for the client-server feature Problem: Not enough testing for the client-server feature. Solution: Add more tests. Add the remote_startserver() function. Fix that a locally evaluated expression uses function-local variables. https://github.com/vim/vim/commit/7416f3e73ab2c4e7ae3adc2ff6e70234f7d40d2e * vim-patch:8.0.0492: a failing client-server request can make Vim hang Problem: A failing client-server request can make Vim hang. Solution: Add a timeout argument to functions that wait. https://github.com/vim/vim/commit/81b9d0bd5c705815e903e671e81b0b05828efd9c Include src/nvim/testdir/test_clientserver.vim changes from patches 8.0.0477, 8.0.0479. * vim-patch:8.0.0633: the client-server test is still a bit flaky Problem: The client-server test is still a bit flaky. Solution: Wait a bit for the GUI to start. Check that the version number can be obtained. https://github.com/vim/vim/commit/60964f68740b8abcbb2d3f0f3aeade21d1bacb22 Include src/nvim/testdir/test_clientserver.vim changes from patches 8.0.0507, 8.0.0511. * vim-patch:8.0.1251: invalid expressin passed to WaitFor() Problem: Invalid expressin passed to WaitFor(). Solution: Check if the variable exists. https://github.com/vim/vim/commit/d97fbf171ec0e63117813da045d2a1c51a9b6f62
* | | Merge #8701 from janlazo/vim-8.0.1140Justin M. Keyes2018-07-12
|\ \ \
| * | | vim-patch:8.0.1387: wordcount test is old styleJan Edmund Lazo2018-07-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Wordcount test is old style. Solution: Change into a new style test. (Yegappan Lakshmanan, closes vim/vim#2434) https://github.com/vim/vim/commit/a703aaee4d6230ed81469ef0cb37f331bf255136
| * | | vim-patch:8.0.1022: test 80 is old styleJan Edmund Lazo2018-07-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Test 80 is old style. Solution: Turn it into a new style test. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/1a333bc44a5d7e51575b1c3cd864fb37fd5b5f2b
| * | | vim-patch:8.0.1253: still too many old style testsJan Edmund Lazo2018-07-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Still too many old style tests. Solution: Convert a few more tests to new style. (Yegappan Lakshmanan, closes vim/vim#2272) https://github.com/vim/vim/commit/430dc5d360166ca5bb6a73f2c87ae53e09282ecb Skip Test_tag_file_encoding. 083_tag_search_with_file_encoding_spec.lua handles the same test case.
| * | | vim-patch:8.0.1140: still old style testsJan Edmund Lazo2018-07-06
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Still old style tests. Solution: Convert two tests to new style. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/bb3e6416f157f68799ccb6070fa2f91df6780e79
* | | | man.vim: C highlighting for EXAMPLES section #8709Synray2018-07-12
| | | |
* | | | Merge pull request #8719 from jamessan/libuv-errnoJames McCoy2018-07-11
|\ \ \ \ | |_|_|/ |/| | | test: Rename includes/pre/uv-errno.h to includes/pre/uv.h
| * | | test/includes: Use ${gen_cdefs} when pre-processing headersJames McCoy2018-07-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that uv.h is directly being included, pre-processing of test/includes/pre/uv.h fails on Linux with In file included from «SRCDIR»/neovim/test/includes/pre/uv.h:1: In file included from /usr/include/uv.h:62: /usr/include/uv/unix.h:72:11: fatal error: 'uv/pthread-barrier.h' file not found # include "uv/pthread-barrier.h" ^~~~~~~~~~~~~~~~~~~~~~ 1 error generated. This happens because we're missing -D_GNU_SOURCE (part of ${gen_cdefs}), which makes the pthread_barrier_* functionality visible.
| * | | test: Rename includes/pre/uv-errno.h to includes/pre/uv.hJames McCoy2018-07-11
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libuv users are only supposed to directly include uv.h. In v1.21.0, all the uv-*.h headers were renamed to uv/*.h, which caused the unit tests to fail with [123/125] Generating post/uv-errno.h FAILED: test/includes/post/uv-errno.h cd «SRCDIR»/src/neovim/build/test/includes && /usr/bin/clang -std=c99 -E -P «SRCDIR»/src/neovim/test/includes/pre/uv-errno.h -I/usr/include -I/usr/include -o «SRCDIR»/neovim/build/test/includes/post/uv-errno.h «SRCDIR»/src/neovim/test/includes/pre/uv-errno.h:1:10: error: 'uv-errno.h' file not found with <angled> include; use "quotes" instead #include <uv-errno.h> ^~~~~~~~~~~~ "uv-errno.h" The intention of the file is to extend libuv's error constants with more values used by the unit tests. This can just as easily be achieved without poking into pseudo-private header files.
* | | Merge pull request #8716 from mhinz/vim-patch/wrapped-linesJames McCoy2018-07-10
|\ \ \ | | | | | | | | [RFC] vim-patch.sh: handle wrapped commit messages
| * | | vim-patch.sh: Unwrap commit messages when reviewing PRsJames McCoy2018-07-10
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Longer summary lines for commits may be wrapped when downloading the commit from GitHub. Unwrapping the lines ensures they can be compared with the raw Vim commits. The difference before and after this patch using: ./scripts/vim-patch.sh -r 8684 Before: ✘ Wrong commit message. Expected: vim-patch:8.0.1464: completing directory after :find does not add slash Problem: Completing directory after :find does not add slash. Solution: Adjust the flags for globpath(). (Genki Sky) vim/vim@8a37b03 Actual: vim-patch:8.0.1464: completing directory after :find does not add slash Problem: Completing directory after :find does not add slash. Solution: Adjust the flags for globpath(). (Genki Sky) After: ✔ Found expected commit message. [ci skip]
* | | vim-patch:8.0.0522: Win32: clipboard=unnamed in :global (#8717)Jan Edmund Lazo2018-07-11
| | | | | | | | | | | | | | | | | | | | | Problem: MS-Windows: when 'clipboard' is "unnamed" yyp does not work in a :global command. Solution: When setting the clipboard was postponed, do not clear the register. https://github.com/vim/vim/commit/3fcfa35f82b9d1ef5e95051b3a45578e10c14ec3
* | | transstr_buf: fix length comparison #8681Michael Hoffmann2018-07-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | closes #8466 closes #8664 Regression by 0d7daaad98d5. - Fix length comparison. - Fix loop(s) which iterated over all fields of array `pcc` even if it was not filled up (try unicode 0x9f as statusline character). Note about the tests: - To input unicode with more than two hex digits you can use <C-v>U...: a + U+fe20: a︠ a + U+fe20 + U+fe21: a︠︡
* | | test: build_stl_str_hl (#8703)ZviRackover2018-07-10
| | | | | | | | | | | | Improve coverage of `build_stl_str_hl`. Minor removal of dead code in the tested function.
* | | vim-patch:8.0.1464: add slash when completing directory #8684Jan Edmund Lazo2018-07-10
| | | | | | | | | | | | | | | Problem: Completing directory after :find does not add slash. Solution: Adjust the flags for globpath(). (Genki Sky) https://github.com/vim/vim/commit/8a37b032895b40dd6953280c33585bcba0c7ef8b
* | | Merge pull request #8708 from blueyed/fix-standoutJames McCoy2018-07-10
|\ \ \ | | | | | | | | Fix standout mode
| * | | [WIP/RFC] Fix standout modeDaniel Hahler2018-07-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was not working for me in different terminals. This patch makes it work in the same way like reverse. Test: :hi jediUsage cterm=standout | hi jediUsage
* | | | Merge pull request #8712 from jamessan/unicode-11James McCoy2018-07-10
|\ \ \ \ | |_|_|/ |/| | | Update unicode files
| * | | Update unicode filesJames McCoy2018-07-10
|/ / /
* | | vim-patch.sh: review_commit: Fix regex for vim versionJames McCoy2018-07-09
| | | | | | | | | | | | | | | 067bb1e9f changed the commit format to include a brief description in the summary line of the commit.
* | | vim-patch.sh: Use single quotes to avoid doubling backslashesJames McCoy2018-07-09
| | |
* | | man.vim: fix for mandoc (#8698)Brayden Banks2018-07-08
| | | | | | | | | | | | | | | When giving a section, the first candidate selection was not performed. followup/fixup #8341
* | | TUI: urxvt: also send xterm focus-reporting seqs #8699Daniel Hahler2018-07-08
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | urxvt supports the xterm sequence for focus reporting in master / CVS since 2016 [1]. This has not been released yet, but is expected to be in 9.23. In 3d0ee17 a special case for rxvt was added, which requires a custom urxvt extension, see #7578. Since urxvt 9.23 is not released still, and extensions for handling this manually appear to be in use, this sends both escape sequences for rxvt. Fixes https://github.com/neovim/neovim/issues/8695. 1: https://github.com/exg/rxvt-unicode/commit/75264fa544529a21ff79d3da26eb3130c43444a5#diff-2c8f5590ce4fa7495edcf7572c89c77b
* | Merge #8702 from janlazo/nvim-8.0.0654Justin M. Keyes2018-07-08
|\ \
| * | tests: endfunc allows uncommented barJan Edmund Lazo2018-07-07
| | |
| * | vim-patch: finish port of 8.0.0{654,663,667}Jan Edmund Lazo2018-07-07
| | | | | | | | | | | | | | | Fix ex_function so that :endfunction passes the test. Remove variables, added in 60c025267265ba4bfc2abd34ea02b13bd5c0e63f.
| * | oldtests: fix func Test_echo_and_string()Jan Edmund Lazo2018-07-07
| |/
* / clint: use stdout for normal/expected output (#8700)Daniel Hahler2018-07-08
|/
* vim-patch:8.0.0663: unexpected error with 'verbose' (#8692)Jan Edmund Lazo2018-07-06
| | | | | Problem: Giving an error message only when 'verbose' set is unexpected. Solution: Give a warning message instead. https://github.com/vim/vim/commit/f8be461d0284110b321be748fea206d4169b98bb
* Merge #8680 from janlazo/vim-8.0.0640Justin M. Keyes2018-07-04
|\
| * vim-patch:8.0.0686: extra redraw when using CTRL-L in second windowJan Edmund Lazo2018-07-03
| | | | | | | | | | | | | | Problem: When typing CTRL-L in a window that's not the first one, another redraw will happen later. (Christian Brabandt) Solution: Reset must_redraw after calling screenclear(). https://github.com/vim/vim/commit/9f5f7bf4d5f757ef885acf74ce03c25429a328aa
| * vim-patch:8.0.0640: mismatch between help and actual messageJan Edmund Lazo2018-07-03
| | | | | | | | | | | | Problem: Mismatch between help and actual message for ":syn conceal". Solution: Change the message to match the help. (Ken Takata) https://github.com/vim/vim/commit/83064068eaabf75a7d235b0eec561dccbcb96b31
* | provider/node: npm --loglevel silent (#8682)Takuya Matsuyama2018-07-04
|/ | | | closes #8674 npm log level may cause unexpected output.
* Merge #8672 'test: nvim_buf_attach after delay'Justin M. Keyes2018-07-02
|\
| * test: nvim_buf_attach: reduce delayJustin M. Keyes2018-07-02
| | | | | | | | | | | | This test is mostly a demo/reference for: https://github.com/neovim/neovim/issues/8634#issuecomment-400940467 so let's not pay a 1s penalty.
| * test: nvim_buf_attach response after initial delayJustin M. Keyes2018-07-01
| | | | | | | | ref #8634