aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | Merge #10708 'remove DYNAMIC_ICONV'Justin M. Keyes2019-08-09
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | closes #3457
| * | | | | remove DYNAMIC_ICONVerw72019-08-06
| | |_|/ / | |/| | |
* | | | | :terminal : update buffer when switching tabpage #10700Adrien Fabre2019-08-09
| | | | | | | | | | | | | | | fix #10696
* | | | | vim-patch:8.1.1540: cannot build without +eval #10729Jan Edmund Lazo2019-08-09
| | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot build without the +eval feature. Solution: Define ex_const if needed. https://github.com/vim/vim/commit/0abb4273f6c14f9ebca8be67d42c6e67b4f612cf
* | | | | test: Minimize shada/helpers.lua #10728Justin M. Keyes2019-08-09
| |/ / / |/| | |
* | | | Merge pull request #10688 from bfredl/utf16libBjörn Linse2019-08-09
|\ \ \ \ | | | | | | | | | | lua: minimal UTF-16 support needed for LSP
| * | | | lua: minimal UTF-16 support needed for LSPBjörn Linse2019-08-08
|/ / / /
* | | | vim-patch:8.1.1824: crash when correctly spelled word is very long (#10725)Jan Edmund Lazo2019-08-08
| | | | | | | | | | | | | | | | | | | | Problem: Crash when correctly spelled word is very long. (Ben Kraft) Solution: Check word length before copying. (closes vim/vim#4778) https://github.com/vim/vim/commit/5bcc5a1ff94bbab1b175e35a72e3df974106b393
* | | | tests: use module pattern with test/functional/helpers.lua (#10724)Daniel Hahler2019-08-08
| | | |
* | | | build: Makefile: use _opt_pylint [ci skip] (#10720)Daniel Hahler2019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | A separate rule it clearer, and allows for `make --keep-going lint` in general later. Ref: https://github.com/neovim/neovim/pull/10714
* | | | tests: output_spec: use shell-test REP_NODELAY (#10726)Daniel Hahler2019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix flaky "shell command :! throttles shell-command output greater than ~10KB:": [ RUN ] shell command :! throttles shell-command output greater than ~10KB: warning: Screen changes were received after the expected state. This indicates indeterminism in the test. Try adding screen:expect(...) (or wait()) between asynchronous (feed(), nvim_input()) and synchronous API calls. - Use screen:redraw_debug() to investigate; it may find relevant intermediate states that should be added to the test to make it more robust. - If the purpose of the test is to assert state after some user input sent with feed(), adding screen:expect() before the feed() will help to ensure the input is sent when Nvim is in a predictable state. This is preferable to wait(), for being closer to real user interaction. - wait() can trigger redraws and consequently generate more indeterminism. Try removing wait(). ERR test/functional/ui/screen.lua:579: Failed to match any screen lines. Expected (anywhere): " %." Actual: |XXXXXXXXXX 591 | |XXXXXXXXXX 592 | |XXXXXXXXXX 593 | |XXXXXXXXXX 594 | | | | | |{3:-- TERMINAL --} | stack traceback: test/functional/ui/screen.lua:579: in function '_wait' test/functional/ui/screen.lua:367: in function 'expect' test/functional/ui/output_spec.lua:63: in function <test/functional/ui/output_spec.lua:53> Log: https://travis-ci.org/neovim/neovim/jobs/569082705#L5355 (gcc-functionaltest-lua)
* | | | build/MSVC: Fix HAVE_ICONV_H #10697erw72019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: HAVE_ICONV_H is not true in MSVC Solution: Since iconv.h does not exist in the standard include directory in MSVC, CMAKE_REQUIRED_INCLUDES must be set. Note: This will be unnecessary after #10708.
* | | | build: port FindLibVterm to LibFindMacros (#10395)Daniel Hahler2019-08-08
| | | |
* | | | build: clean up / remove X_USE_STATIC (#10713)Daniel Hahler2019-08-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was discouraged (as an option) in 5b5d353151 [1], not enabled/used by default, and not working according to the comment in local.mk.example. Taken out of https://github.com/neovim/neovim/pull/10395. 1: https://github.com/neovim/neovim/pull/2465
* | | | ci: AppVeyor: fix upload of coverage for oldtest (#10721)Daniel Hahler2019-08-07
| |/ / |/| | | | | | | | | | | This was not working due to having another `python` in the PATH then. Ref: https://ci.appveyor.com/project/neovim/neovim/builds/26492761/job/dspm40v5l2v6gn40?fullLog=true#L15955
* | | Fix lualint: remove unused varDaniel Hahler2019-08-07
| | |
* | | cmakelists: fixed tests to avoid clang warnings (#10705)Ihor Antonov2019-08-07
| | | | | | | | | | | | * clang/"dead assignment" * clang/"Uninitialized argument value"
* | | tests: more cleanup of plugin/shada_specDaniel Hahler2019-08-07
| | | | | | | | | | | | | | | Ref: https://github.com/neovim/neovim/pull/10701/commits/330a6713#r311005754 Closes https://github.com/neovim/neovim/pull/10710.
* | | build: lint: fix exit with optional pylintDaniel Hahler2019-08-07
| | | | | | | | | | | | Ref: https://github.com/neovim/neovim/pull/10714#issuecomment-518932355
* | | Merge pull request #10703 from ngortheone/clang_dead2Björn Linse2019-08-07
|\ \ \ | | | | | | | | | | | | | | | | | | | | clang/"dead assignment": suppessed getchar.c Possibly invalid, read on line 1972: if (keylen >= 0 && keylen <= typebuf.tb_len) {
| * | | clang/"dead assignment": suppessedIhor Antonov2019-08-05
| | | | | | | | | | | | | | | | | | | | To ensure consistency and avoid possible errors in the code the assigment is considered to be legit.
* | | | Merge pull request #10513 from bfredl/bytecountBjörn Linse2019-08-06
|\ \ \ \ | | | | | | | | | | api/lua: add {byte_count} parameter to line region change event
| * | | | lua: support getting UTF-32 and UTF-16 sizes of replaced textBjörn Linse2019-08-06
| | | | |
| * | | | lua: add {old_byte_size} to on_lines buffer change eventBjörn Linse2019-08-06
| | |/ / | |/| |
* | | | Fix list_features to include space after first feature (#10711)Daniel Hahler2019-08-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Regressed in e134cc9d4a: the use of list_in_columns was not adding a space after the first features, because we do not start on a new line: > Features: -acl+iconv +tui This moves all the related code to `list_features`, and just joins them with spaces.
* | | | build: move pylint to Makefile, optional with "make lint" (#10714)Daniel Hahler2019-08-06
| | | | | | | | | | | | | | | | | | | | This avoids errors when using "make lint", but "flake8" is not available. We do not want to install it then via third-party. On CI "make pylint" is used explicitly.
* | | | tests: unit.helpers: provide string with write errors (#10715)Daniel Hahler2019-08-06
|/ / / | | | | | | | | | This might help to have more information in case of errors, like mentioned in https://github.com/neovim/neovim/commit/eec529cf9e.
* | | Merge #10701 from justinmk/test-fixesJustin M. Keyes2019-08-06
|\ \ \ | | | | | | | | test: Eliminate plugin/helpers.lua
| * | | test/mbyte_spec: skip broken test on QuickBuildJustin M. Keyes2019-08-06
| | | | | | | | | | | | | | | | Forgot `return` in eec529cf9e13.
| * | | test: Eliminate plugin/helpers.luaJustin M. Keyes2019-08-05
| | | |
* | | | Merge #10655 'environ(), getenv(), setenv()'Justin M. Keyes2019-08-06
|\ \ \ \ | | | | | | | | | | | | | | | close #10655
| * | | | f_environ: cleanup/refactorDaniel Hahler2019-08-06
| | | | | | | | | | | | | | | | | | | | | | | | | - use os_getenvname_at_index / os_getenv - f_getenv: empty (*p == NUL) is not null (undefined)
| * | | | vim-patch:8.1.1305: there is no easy way to manipulate environment variablesDaniel Hahler2019-08-06
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: There is no easy way to manipulate environment variables. Solution: Add environ(), getenv() and setenv(). (Yasuhiro Matsumoto, closes vim/vim#2875) https://github.com/vim/vim/commit/691ddeefb545d8488e5a495af61caba2e57b3de9
* / / / clang/"dead assignment": screen.c #10702Ihor Antonov2019-08-06
|/ / / | | | | | | | | | | | | | | | | | | Suppress the warning. mb_c and mb_l describe a char together, they are not independent variables. The coupled assignments are good practice to avoid future confusion, even if the current code doesn't use an assigned value.
* / / vim-patch:8.1.1439: ga_grow(): 1.5x growth rate #10699Jan Edmund Lazo2019-08-05
|/ / | | | | | | | | | | Problem: Json_encode() is very slow for large results. Solution: In the growarray use a growth of at least 50%. (Ken Takata, closes vim/vim#4461) https://github.com/vim/vim/commit/c47ed44be76a520ded90913099771999c8a79eeb
* | Merge pull request #10690 from bfredl/lua_printBjörn Linse2019-08-05
|\ \ | | | | | | lua: laundry list (crashes and additions)
| * | lua: add vim.in_fast_event() to check if we are in a luv callbackBjörn Linse2019-08-05
| | |
| * | lua: do not crash on syntax error in debug.debug()Björn Linse2019-08-05
| | |
| * | lua: immediate-callback safe print()Björn Linse2019-08-05
| | |
* | | Merge #10695 from justinmk/test-fixesJustin M. Keyes2019-08-05
|\ \ \ | | | | | | | | test: various fixes
| * | | test/man_spec: remove plugin_helpers.reset()Justin M. Keyes2019-08-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The call to plugin_helpers.reset() is redundant with the clear() call above it. Probably just a copy-paste mistake. Avoids exit_event race #8813. Helped-by: Björn Linse <bjorn.linse@gmail.com>
| * | | test/mbyte_spec: skip broken test on QuickBuildJustin M. Keyes2019-08-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 14:13:04,119 INFO - # ./test/unit/helpers.lua @ 760: mbyte utf_char2bytes for chars 0xa000 - 0xafff 14:13:06,307 WARN - E908: using an invalid value as a String /usr/home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/.deps/usr/bin/luajit: ./test/unit/helpers.lua:459: write() error: 32: Broken pipe 14:13:06,308 WARN - stack traceback: 14:13:06,308 WARN - [C]: in function 'throw' 14:13:06,308 WARN - ...quests-automated/.deps/usr/share/lua/5.1/busted/core.lua:149: in function 'error' 14:13:06,308 WARN - ...ts-automated/.deps/usr/share/lua/5.1/luassert/assert.lua:171: in function 'assert' 14:13:06,308 WARN - ./test/unit/helpers.lua:459: in function 'write' 14:13:06,308 WARN - ./test/unit/helpers.lua:626: in function 'hook' 14:13:06,308 WARN - ./test/unit/helpers.lua:574: in function <./test/unit/helpers.lua:557> 14:13:06,308 WARN - [C]: in function 'type' 14:13:06,308 WARN - ...d/.deps/usr/share/lua/5.1/busted/outputHandlers/base.lua:57: in function 'copyElement' 14:13:06,308 WARN - ...d/.deps/usr/share/lua/5.1/busted/outputHandlers/base.lua:66: in function 'format' 14:13:06,308 WARN - ...d/.deps/usr/share/lua/5.1/busted/outputHandlers/base.lua:172: in function 'fn' 14:13:06,308 WARN - ...-requests-automated/.deps/usr/share/lua/5.1/mediator.lua:103: in function 'publish' 14:13:06,308 WARN - ...quests-automated/.deps/usr/share/lua/5.1/busted/core.lua:201: in function 'safe' 14:13:06,308 WARN - ...quests-automated/.deps/usr/share/lua/5.1/busted/core.lua:312: in function 'execute' 14:13:06,308 WARN - ...sts-automated/.deps/usr/share/lua/5.1/busted/execute.lua:58: in function 'execute' 14:13:06,308 WARN - ...ests-automated/.deps/usr/share/lua/5.1/busted/runner.lua:197: in function <...ests-automated/.deps/usr/share/lua/5.1/busted/runner.lua:11> 14:13:06,308 WARN - ./.deps/usr/lib/luarocks/rocks/busted/2.0.0-1/bin/busted:3: in main chunk 14:13:06,308 WARN - [C]: at 0x004041a0 14:13:06,323 WARN - Terminated 14:13:06,325 INFO - Executing post-execute action... 14:13:06,526 INFO - Checking step execute condition... 14:13:06,526 INFO - Step execute condition satisfied, executing... 14:13:06,706 INFO - Executing pre-execute action... 14:13:06,706 INFO - Running step...
| * | | test: isCI(): add "name" parameterJustin M. Keyes2019-08-05
| | | |
| * | | tests: fix flaky "TermClose … fast-exiting terminal job stops"Justin M. Keyes2019-08-05
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This extra retry() was removed (at my suggestion) in 5b94a2977a22, but it is probably needed: jobwait(…, timeout=0) could return while channel_process_exit_cb() is still queued (so TermClose event didn't fire yet). 20:46:21,288 INFO - not ok 547 - TermClose event triggers when fast-exiting terminal job stops 20:46:21,288 INFO - # test/functional/autocmd/termclose_spec.lua @ 20 20:46:21,288 INFO - # Failure message: ./test/functional/helpers.lua:98: Vim:E121: Undefined variable: g:test_termclose 20:46:21,288 INFO - # stack traceback: 20:46:21,288 INFO - # ./test/functional/helpers.lua:98: in function 'eval' 20:46:21,288 INFO - # test/functional/autocmd/termclose_spec.lua:25: in function <test/functional/autocmd/termclose_spec.lua:20>
* | | vim-patch:8.1.1383: warning for size_t/int mixup (#10694)Jan Edmund Lazo2019-08-05
| | | | | | | | | | | | | | | Problem: Warning for size_t/int mixup. Solution: Change type. (Mike Williams) https://github.com/vim/vim/commit/d33a764123a8aedb20cd84aeff3b94810ee67c4c
* | | :doautocmd : Never show "No matching autocommands" #10689Justin M. Keyes2019-08-05
| | | | | | | | | | | | | | | | | | | | | | | | The message is useless, it doesn't even mention the event name. vim_dev discussion: https://groups.google.com/forum/#!msg/vim_dev/RTbq58TMq9w/Xr4rSoUTCgAJ ref: https://github.com/vim/vim/issues/4300
* | | vim-patch:8.1.1311: test: abort autocmd with exception #10692Jan Edmund Lazo2019-08-05
| | | | | | | | | | | | | | | | | | Problem: Aborting an autocmd with an exception is not tested. Solution: Add a test. Also shows how to abort a command by throwing an exception. https://github.com/vim/vim/commit/23b5139234a79567097ca73aba15ea134381b934
* | | vim-patch:8.1.1251: test completion of mapping keys #10691Jan Edmund Lazo2019-08-05
|/ / | | | | | | | | Problem: No test for completion of mapping keys. Solution: Add a test. Also clean up the code. https://github.com/vim/vim/commit/2cb9f0253228478ec27265ff40f59c629874d695
* | Makefile: only use pattern rules with BUILD_TYPE=Ninja (#10687)Daniel Hahler2019-08-04
| | | | | | | | | | | | While not doing any harm with "Unix Makefiles", they do not work there as-is. Therefore just do not use them then. Followup to #10366 (7f6ff829a).
* | runtime/matchit.vim: workaround broken 'packpath'Justin M. Keyes2019-08-04
| | | | | | | | fix #10680
generated by cgit (git 2.34.1) at 2025-07-19 22:58:28 +0000