aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | headerDaniel Hahler2019-08-07
| | | |
| * | | orig src/nvim/change.cDaniel Hahler2019-08-07
| | | |
| * | | vim-patch:3f86ca0faa29cb862f876a97f87790f3a46a3858Daniel Hahler2019-08-07
| | | | | | | | | | | | | | | | | | | | Add missing files from patch 8.1.1318 https://github.com/vim/vim/commit/3f86ca0faa29cb862f876a97f87790f3a46a3858
| * | | vim-patch:8.1.1318: code for text changes is in a "misc" fileDaniel Hahler2019-08-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Code for text changes is in a "misc" file. Solution: Move the code to change.c. https://github.com/vim/vim/commit/ec28d1516eb8bb5dcaa42de145953a6d49aebb6f
* | | | channels: reflect exit due to signals in exit status code (#10573)Daniel Hahler2019-08-09
| | | | | | | | | | | | | | | | | | | | Uses `128 + term_signal` in case of exit due to a signal. Fixes https://github.com/neovim/neovim/issues/10571.
* | | | tests/functional: expect_msg_seq: use load_adjust (#10727)Daniel Hahler2019-08-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Regardless of the comment "Big timeout for ASAN/valgrind" it would use 10s by default already. This changes it to use `load_adjust`, which itself is only computed on CI now, and outside of any tests - since it has side effects when being used the first time! The failure seen on AppVeyor: [ RUN ] jobs can get the pid value using getpid: ERR test\functional\helpers.lua:167: ============================================================================== got 0 messages (ignored 0), expected 1 stack traceback: test\functional\helpers.lua:167: in function 'expect_msg_seq' test/functional\core\job_spec.lua:288: in function <test/functional\core\job_spec.lua:281> Log: https://ci.appveyor.com/project/neovim/neovim/builds/26537324/job/y1io66fbx399q7h6?fullLog=true#L6554
* | | | Merge #9782 from janlazo/vim-8.1.0199Justin M. Keyes2019-08-09
|\ \ \ \ | | | | | | | | | | vim-patch:8.1.{199,200}
| * | | | f_spellbadword: set len=0 for non-found wordDaniel Hahler2019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `len` is used with `list_append_string` later, and should reflect the length of `word` (i.e. 0 when not setting word / breaking above). Ref: neovim/neovim#9782 (comment)
| * | | | vim-patch:8.1.0200: spellbadword() not testedJan Edmund Lazo2019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: spellbadword() not tested. Solution: Add a test. (Dominique Pelle, closes vim/vim#3235) https://github.com/vim/vim/commit/872e451e8c326d5dd3062ef621fcbf0a4c5bef78
| * | | | vim-patch:8.1.0199: spellbadword() does not check for caps errorJan Edmund Lazo2019-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: spellbadword() does not check for caps error. (Dominique Pelle) Solution: Adjust capcol when advancing. https://github.com/vim/vim/commit/66ab916935585391b2efaa8e39075e1ef94717b1
* | | | | clang/"Null passed as a nonnull parameter" #10718Ihor Antonov2019-08-09
| | | | |
* | | | | 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
| | |