aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
* Merge pull request #10541 from bfredl/conversion_getlnBjörn Linse2019-07-19
|\ | | | | refactor: enable -Wconversion for ex_getln.c and use int for Rows/Columns
| * refactor: use int for Columns and RowsBjörn Linse2019-07-19
| |
| * refactor: enable -Wconversion for ex_getln.cBjörn Linse2019-07-19
| |
* | PVS/V1026: possible overflow in a loop #10529Ihor Antonov2019-07-19
| | | | | | | | | | | | * pvs/V1026: possible overflow in a loop * refactor: char_u -> uint8_t * lint * remove unused if branch: enc_utf8 is always true
* | Merge #10543 'PVS/V560'Justin M. Keyes2019-07-19
|\ \ | |/ |/|
| * lintIhor Antonov2019-07-19
| |
| * pvs/V560: A part of conditional expression is always trueIhor Antonov2019-07-19
| | | | | | | | wp->w_p_list is checked earlier in the same composite if condition
| * pvs/V560: A part of conditional expression is always trueIhor Antonov2019-07-19
|/ | | | c == '\000' is always true because of the if branch it is already in
* option_defs.h: fix incorrect definition #10542Justin M. Keyes2019-07-19
|
* test: Force $TEST_FILE to relative path [ci skip] #10535erw72019-07-19
| | | | | - Always force TEST_PATH to a relative path. - Eliminate rel_test_path. (The code simplification is worth the extra subdirectory below Xtest_tmpdir/.)
* vim-patch:8.1.0740: Tcl test fails (#10540)Jan Edmund Lazo2019-07-19
| | | | | | Problem: Tcl test fails. Solution: When the argument is empty don't give an error, instead rely on the error reporting higher up. https://github.com/vim/vim/commit/8309b0559da6e9a581a7816572594c90c2d7f942
* Merge #10368 from janlazo/vim-8.0.1164Justin M. Keyes2019-07-19
|\ | | | | vim-patch:8.0.{755,1146,1164}
| * vim-patch:8.0.1164: changing StatusLine highlight does not always workJan Edmund Lazo2019-07-06
| | | | | | | | | | | | | | | | Problem: Changing StatusLine highlight while evaluating 'statusline' may not change the status line color. Solution: When changing highlighting while redrawing don't cause another redraw. (suggested by Ozaki Kiichi, closes vim/vim#2171, closes vim/vim#2120) https://github.com/vim/vim/commit/65ed136844fbaffdd473903ed841c944600234dc
| * vim-patch:8.0.1146: redraw when highlight is set with same namesJan Edmund Lazo2019-07-06
| | | | | | | | | | | | Problem: Redraw when highlight is set with same names. (Ozaki Kiichi) Solution: Only free and save a name when it changed. (closes vim/vim#2120) https://github.com/vim/vim/commit/452030e530aad9b08fcfa71737d098b33c752b85
| * vim-patch:8.0.0755: terminal window does not have colors in the GUIJan Edmund Lazo2019-07-06
| | | | | | | | | | | | Problem: Terminal window does not have colors in the GUI. Solution: Lookup the GUI color. https://github.com/vim/vim/commit/26af85d97ba1ed0ade6cdd41890ca04ed879b9c7
* | Merge #10052 from janlazo/vim-8.1.0729Justin M. Keyes2019-07-19
|\ \ | | | | | | vim-patch:8.1.{729,732}
| * | vim-patch:8.1.0732: cannot build without the eval featureJan Edmund Lazo2019-07-17
| | | | | | | | | | | | | | | | | | Problem: Cannot build without the eval feature. Solution: Make a copy of the sourced file name. https://github.com/vim/vim/commit/ea56e167c87352f07a77d3661425e336817a7141
| * | vim-patch:8.1.0729: there is a SourcePre autocommand event but not a SourcePostJan Edmund Lazo2019-07-17
| | | | | | | | | | | | | | | | | | Problem: There is a SourcePre autocommand event but not a SourcePost. Solution: Add the SourcePost autocommand event. (closes vim/vim#3739) https://github.com/vim/vim/commit/2b6185287adf53343ed5f49e967ae402c64063e4
* | | Revert "Downgrade to clang-4.0 to avoid false-positive warnings from clang" ↵Daniel Hahler2019-07-18
| | | | | | | | | | | | | | | [skip appveyor] (#10487) This reverts commit 2cbac719c3eba8ea5826e16912126d70222911ed.
* | | pvs/V1037: two case branches doing the same thing (#10527)Ihor Antonov2019-07-18
| | |
* | | PVS/V1037: suppress warning #10526Ihor Antonov2019-07-18
| | | | | | | | | | | | * Case branches are semantically different. * Lint