aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | tests: runnvim.vim: do not call jobstop() (#10659)Daniel Hahler2019-07-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It should be done for timeouts only (-1, not != -1), but the job is stopped via `:bwipeout!` already also in that case. Fixes E900 with timing out tests: "test_xxx.tlog" "test_xxx.tlog" [New] 183L, 13375C written Error detected while processing function Main: line 37: E900: Invalid channel id================================================================================
* | | | Makefile: use pattern rules for build/.deps [ci skip] (#10366)Daniel Hahler2019-07-30
| | | | | | | | | | | | | | | | This allows for `make build/bin/nvim`, `make build/src/nvim/auto/funcs.generated.h` etc.
* | | | Merge pull request #10656 from blueyed/minorDaniel Hahler2019-07-30
|\ \ \ \ | | | | | | | | | | Minor: align test, fix comment/usage
| * | | | gen_eval.lua: use correct name in usageDaniel Hahler2019-07-30
| | | | |
| * | | | tests: use Vim's version for patch 8.1.0005Daniel Hahler2019-07-30
|/ / / / | | | | | | | | | | | | | | | | Not sure what happened in daca22f5b / #8821, but Vim used assert_match / no expand in vim/vim@d19b23415 (8.1.0005) already.
* | | | vim-patch: handle tags, pass through git-log options (#10140)Daniel Hahler2019-07-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * scripts/vim-patch.sh: fall back to "origin" for remote Without this, it would fail e.g. with a locally cloned repo of Neovim. * scripts/vim-patch.sh: assign_commit_details: handle tags [ci skip] - Handle "v" prefix from Vim tags. - Exit in case of error therein already. * -l/-L: pass through git-log options [ci skip] This allows for only listing missing patches for a given Vim file: > scripts/vim-patch.sh -L src/edit.c
* | | | Merge pull request #10648 from blueyed/autopep8Daniel Hahler2019-07-30
|\ \ \ \ | |/ / / |/| | | ci: py: flake8 fixes
| * | | ci: pylint target via flake8Daniel Hahler2019-07-29
| | | | | | | | | | | | | | | | Ref: https://github.com/neovim/neovim/pull/10631#issuecomment-515756591
| * | | py: flake8 fixesDaniel Hahler2019-07-29
| | | |
| * | | scripts: autopep8Daniel Hahler2019-07-29
|/ / /
* | | Merge #10645 from janlazo/vim-8.1.0999Justin M. Keyes2019-07-29
|\ \ \ | |/ / |/| | vim-patch:8.1.{1005,1041,1049,1052,1053,1086,1394}
| * | vim-patch:8.1.1086: too many curly bracesJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Too many curly braces. Solution: Remove curly braces where they are not needed. (Hirohito Higashi, closes vim/vim#3982) https://github.com/vim/vim/commit/abab0b0fdd6535969447b03a4fffc1947918cf6c Neovim code style requires the opposite. Add curly braces to minimize lint errors when applying Vim patches.
| * | buffer: add attributes to pure functionsJan Edmund Lazo2019-07-28
| | |
| * | vim-patch:8.1.1049: when user tries to exit with CTRL-C message is confusingJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When user tries to exit with CTRL-C message is confusing. Solution: Only mention ":qa!" when there is a changed buffer. (closes vim/vim#4163) https://github.com/vim/vim/commit/a84a3dd6635fcd2e07f510cba6a999585dcc381a vim-patch:8.1.1052: test for CTRL-C message sometimes fails Problem: test for CTRL-C message sometimes fails Solution: Make sure there are no changed buffers. https://github.com/vim/vim/commit/553e5a5c568e7d175b65b0472cd6d9843b25f4c8 vim-patch:8.1.1053: warning for missing return statement Problem: Warning for missing return statement. (Dominique Pelle) Solution: Add return statement. https://github.com/vim/vim/commit/d6c3f1fa2b5e1dd7dc87cf608d72b84ad696b58f
| * | vim-patch:8.1.1041: test for Arabic no longer neededJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | Problem: Test for Arabic no longer needed. Solution: Remove the test for something that was intentionally left out. https://github.com/vim/vim/commit/295e3ba31f76818f195c654df506c88cb65f9289
| * | vim-patch:8.1.1394: not restoring t_F2 in registers testJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | Problem: Not restoring t_F2 in registers test. Solution: Assign to &t_F2 instead of t_F2. (Andy Massimino, closes vim/vim#4434) https://github.com/vim/vim/commit/a5e44600ebb02057fb15ae7a2f15c5cae8f63bcc
| * | vim-patch:8.1.1005: test fails because t_F2 is not setJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | Problem: Test fails because t_F2 is not set. Solution: Add try-catch. https://github.com/vim/vim/commit/55d81cd2a15d93dba89b2ff1c988d82789919b6b
* | | Merge #10615 'vim-patch:8.1.0053'Justin M. Keyes2019-07-29
|\ \ \ | | | | | | | | fix #10604
| * | | vim-patch:8.1.0053 use typval_T in the caller of call_vim_functioncangscop2019-07-29
| | | | | | | | | | | | | | | | | | | | | | | | Problem: unreliable types for complete function arguments Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi) vim/vim#2993
| * | | vim-patch:8.1.0053 use typval_T in the caller of call_vim_functioncangscop2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Problem: unreliable types for complete function arguments Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi) vim/vim#2993
| * | | vim-patch:8.1.53 use typval_T in the caller of call_vim_functioncangscop2019-07-27
| | | | | | | | | | | | | | | | | | | | | | | | Problem: unreliable types for complete function arguments Solution: fix argument type for functions w/ unreliable type conversion(Ozaki Kiichi) vim/vim#2993
* | | | PVS/V507: suppress false positive #10647Ihor Antonov2019-07-29
| | | | | | | | | | | | | | | | The pointer does not outlive the value in local scope as it gets reassigned immediately after scope.
* | | | build/macOS: enable fallthrough attribute #10653Gavin D. Howard2019-07-29
| |/ / |/| |
* | | Merge #10641 from ngortheone/pvs/V560_6Justin M. Keyes2019-07-29
|\ \ \ | | | | | | | | pvs/V560: part of conditional expression is always false
| * | | lint: makemapIhor Antonov2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * if braces * spaces * post increments * comment style
| * | | pvs/V560: part of conditional expression is always falseIhor Antonov2019-07-28
|/ / / | | | | | | | | | makemap: same condition is used to skip outer loop cycle
* | | vim-patch:8.1.1138: add CompleteChanged #10644Justin M. Keyes2019-07-29
| | | | | | | | | | | | | | | | | | (This was originally a Neovim patch, but this commit merges some changes from the Vim patch.) https://github.com/vim/vim/commit/d7f246c68cfb97406bcd4b098a2df2d870b3ef92
* | | vim-patch:8.1.0017: shell command completion has duplicates #10616erw72019-07-29
| | | | | | | | | | | | | | | Problem: Shell command completion has duplicates. (Yegappan Lakshmanan) Solution: Use a hash table to avoid duplicates. (Ozaki Kiichi, closes vim/vim#539, closes vim/vim#2733) https://github.com/vim/vim/commit/62fe66f251263715968442e237742d9d3dfd5fa1
* | | clang/"dead assignments" #10620Bartosz Miera2019-07-29
| | | | | | | | | | | | Also revert possibly-wrong change in f_sign_getplaced() from 652be3cb0040.
* | | TextYankPost: spurious/too-early dispatch during delete #10392Jit2019-07-29
| | | | | | | | | | | | | | | | | | Problem: delete-with-register dispatches TextYankPost before updating yank registers Solution: Add flag to op_yank(). Fixes #10225
* | | Merge #10643 from janlazo/vim-8.1.1765Justin M. Keyes2019-07-29
|\ \ \ | | | | | | | | vim-patch:8.1.{990,992,1765}
| * | | vim-patch:8.1.0990: floating point exception with "%= 0" and "/= 0"Jan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Floating point exception with "%= 0" and "/= 0". Solution: Avoid dividing by zero. (Dominique Pelle, closes vim/vim#4058) https://github.com/vim/vim/commit/e21c1580b7acb598a6e3c38565434fe5d0e2ad7a
| * | | vim-patch:8.1.1765: get(func, dict, def) does not work properlyJan Edmund Lazo2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Problem: get(func, dict, def) does not work properly. Solution: Handle NULL dict better. (Takuya Fujiwara, closes vim/vim#4734) https://github.com/vim/vim/commit/f91aac5e3e3b8b1633d84eac2687ebbd76d8133b
* | | | PVS/V547: expression is always true/false #10640Ihor Antonov2019-07-29
| | | | | | | | | | | | | | | | Functions marked with FUNC_ATTR_NONNULL_RET do not return NULL. Remove redundant checks.
* | | | PVS/V512: memcpy overflow/underflow #10642Ihor Antonov2019-07-28
|/ / / | | | | | | | | | | | | problem: bfredl: pvs thinks the type of the pointed at item is too small. solution: refactored address calculation.
* | | PVS/V560: condition is always false #10638Ihor Antonov2019-07-28
| | | | | | | | | * shorten_buf_name: condition inside outer if-branch with inverse condition
* | | vim-patch:8.0.1753: fix various warnings #10639Ihor Antonov2019-07-28
| | | | | | | | | | | | | | | | | | vim-patch:8.0.1753: various warnings from a static analyser Problem: Various warnings from a static analyser Solution: Remove unneeded conditions. https://github.com/vim/vim/commit/1c17ffa4611f4efe68c61f7cdd9ed692a866ba75
* | | context: shada_encode_regs(): init WriteMergerState #10637Justin M. Keyes2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Check shada_pack_xx() result, abort on failure. - Use xcalloc() to zero-initialize `wms`, fixes below ASAN failure. ASAN failure (running vim_spec.lua in a loop): ../src/nvim/shada.c:1773:13: runtime error: load of value 224, which is not a valid value for type 'bool' SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../src/nvim/shada.c:1773:13 Adding an explicit check just before the ASAN failed line: case kSDItemRegister: { if (*((int *)(void *)&entry.data.reg.is_unnamed) != 1 && *((int *)(void *)&entry.data.reg.is_unnamed) != 0) { abort(); (gdb) p entry.data.reg +p entry.data.reg $5 = {name = 49 '1', type = kMTLineWise, contents = 0x60200000c250, is_unnamed = true, contents_size = 1, width = 0, additional_data = 0x0} (gdb) p *((uint8_t *)(void *)&entry.data.reg.is_unnamed) +p *((uint8_t *)(void *)&entry.data.reg.is_unnamed) $7 = 2049
* | | PVS/V560: expression is always false/true #10623Ihor Antonov2019-07-28
| | |
* | | vim-patch:7.4.2213: runtime parts with EndOfBuffer port #10635Daniel Hahler2019-07-28
| | | | | | | | | | | | | | | | | | | | | Problem: Cannot highlight the "~" lines at the end of a window differently. Solution: Add the EndOfBuffer highlighting. (Marco Hinz, James McCoy) https://github.com/vim/vim/commit/58b853460add42098ab08017df9e030fb14fd34b Includes minor doc parts from 7.4.1304 for runtime/doc/eval.txt.
* | | ci: Travis: no need for asan_symbolize (#10627)Daniel Hahler2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Having llvm-symbolizer in the $PATH is enough. - check_logs: remove log after displaying it Otherwise it would be displayed/symbolized again and again. E.g. in https://api.travis-ci.org/v3/job/564477704/log.txt.
* | | src/clint.py: flake8 fixes [ci skip] (#10631)Daniel Hahler2019-07-28
| | |
* | | vim-patch:b1c9198af (#10634)Daniel Hahler2019-07-28
| | | | | | | | | | | | | | | | | | Vim 8.1 release Update version number and information. Fix a couple of tests. https://github.com/vim/vim/commit/b1c9198afb7ff902588b45fbe44f0760a9f48375
* | | Merge #10628 from notomo/vim-8.1.1759Justin M. Keyes2019-07-28
|\ \ \
| * | | Fix clint errornotomo2019-07-28
| | | |
| * | | vim-patch:8.1.1759: no mode char for terminal mapping from maparg()notomo2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Problem: No mode char for terminal mapping from maparg(). Solution: Check for TERMINAL mode. (closes vim/vim#4735) https://github.com/vim/vim/commit/14371ed69778107654e39268d0d90982e53ad6e0
* | | | build/tests: remove pre/uv.h #10531Daniel Hahler2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | Initially done in 28e59cb22, but does not appear to be necessary anymore. Uses UV_EEXIST directly, just like UV_ENOENT.
* | | | PVS/V560: condition is always true #10630Ihor Antonov2019-07-28
| | | | | | | | | | | | earlier condition leads to short-circuit of the loop
* | | | PVS/V560: condition is always true #10624Ihor Antonov2019-07-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * before changed line status variable is set to RA_FAIL in 2 places line 3816 and 3826, both lead to short-circuit of the loop, making comparing it to RA_FAIL useless close #10624
* | | | pvs/V560: part of conditional expression is always true (#10629)Ihor Antonov2019-07-28
|/ / / | | | | | | | | | Since timed_out is pointer to int condition timed_out != NULL && timed_out makes no sense as both are effectively the same thing. Most likely the first check was meant to guard the dereference of the second. Wrong import of Vim patch (8.0.0643).