aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* pvscheck: Provide arguments to `patch_sources` in correct orderZyX2017-05-20
|
* options: Silence V542 without using commentsZyX2017-05-20
|
* options: Silence V542 the other wayZyX2017-05-20
| | | Still does not work though.
* syntax: Silence V782ZyX2017-05-20
| | | | Just another pointer hack used with hash tables.
* tui: Fix V547: always true conditionZyX2017-05-20
| | | | | The variable in question is initalized at the start of the function with something non-NULL, specifically pointer to a static buffer.
* spellfile: Fix V547: always true conditionZyX2017-05-20
| | | This condition was already checked at the surrounding if() at line 2422.
* tag: Silence V522: potential null dereferenceZyX2017-05-20
| | | | | Call PVS is referring to is using DT_FREE which will make function exit earlier, in #ifdef EXITFREE block.
* shada: Fix V581: adjacent branches with same conditionZyX2017-05-20
|
* search: Fix V502: ?: ambiguityZyX2017-05-20
|
* options: Silence V542: odd casts for .def_valZyX2017-05-20
|
* memfile: Fix V547: always true conditionZyX2017-05-20
| | | | | `blocksize` was checked against UINT_MAX after it was checked against MAX_SWAP_PAGE_SIZE which makes it always pass the check. Better use STATIC_ASSERT instead.
* main: Silence V522: potential NULL pointer dereferenceZyX2017-05-20
| | | | | | | | AFAIK there is no way NULL can be there, including from the line it points to. Dunno what analyser was thinking, but dereferencing of `argv[0]` happened just before `get_number_arg()` call: in `ascii_isdigit()` two lines above. And `idx` cannot possibly be NULL ever, it comes from `&varname`, this could not ever give anything, but a valid pointer.
* getchar: Eliminate two-iteration loopZyX2017-05-20
|
* eval: Silence V614: potentially uninitialized variableZyX2017-05-20
| | | | | Could not be uninitialized because `func_or_func_caller_profiling` is true only if `do_profiling` is `YES`, and if `do_profiling` is `YES` then `script_prof_save()` was called to initialize the variable.
* eval: Fix V507ZyX2017-05-20
|
* eval: Silence V782ZyX2017-05-20
|
* eval/encode: Silence V595 errorZyX2017-05-20
|
* pvscheck: Handle invalid option error gracefullyZyX2017-05-20
|
* pvscheck: Add --only-analyse modeZyX2017-05-20
|
* buffer: Silence V519ZyX2017-05-20
|
* edit: Silence V595ZyX2017-05-20
|
* *: Fix all V641 errorsZyX2017-05-20
|
* charset: Fix V695: dead branchesZyX2017-05-20
| | | | | | | | Based on comments it appears that some non-printable characters intended to be shown as `|x` (0xA0..0xFE) and some as `~x` (0x80..0x9F, 0xFF, excluding previous). But this never happens because this is being catched by condition `c >= 0x80` above which makes them be represented as `<A0>`. Since I find this variant more useful and it additionally is backwards compatible (Vim does the same thing) I just dropped dead branches.
* man.vim: check for -l flag #6766raichoo2017-05-19
|
* startup: init v:progpath before calling vim_getenv (#6755)Justin M. Keyes2017-05-17
|
* Merge pull request #6759 from jamessan/flaky-timersJames McCoy2017-05-16
|\ | | | | oldtests: Mark test_timer's Test_oneshot as flaky
| * oldtests: Mark test_timer's Test_oneshot as flakyJames McCoy2017-05-16
|/
* Merge #6741 from justinmk/progpathJustin M. Keyes2017-05-15
|\
| * env_iter: Learn `delim` parameter.Justin M. Keyes2017-05-15
| |
| * vim_getenv: Use v:progpath instead of os_exepath.Justin M. Keyes2017-05-15
| |
| * startup: v:progpath fallback: path_guess_exepathJustin M. Keyes2017-05-15
| | | | | | | | | | | | | | If procfs is missing then libuv cannot find the exe path. Fallback to path_guess_exepath(), adapted from Vim findYourself(). Closes #6734
* | Merge #6737 "options: make 'highlight' read-only"Justin M. Keyes2017-05-15
|\ \ | |/ |/|
| * options: make 'highlight' read-onlyBjörn Linse2017-05-15
| |
* | ci: Enable staged builds (#6739)James McCoy2017-05-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This should better allow distributing the load among PRs, while getting critical feedback to the submitter sooner. First stage runs the ASAN/UBSAN/TSAN since any failures in those are gating issues. Second stage runs the rest of the normal builds in parallel. Remaining stages provide lower priority feedback. The lint build runs fast locally, so it's better to run that locally than wait on CI. The coverage build is pretty fickle, so it is only run once all other jobs are green.
* | Merge #6480 from ZyX-I/colored-cmdline'/input-dictJustin M. Keyes2017-05-13
|\ \
| * | doc: Add a note to vim_diff.txtZyX2017-05-13
| | |
| * | functests: Reword regression test headersZyX2017-05-13
| | |
| * | functests: Remove “correctly” from non-regression testsZyX2017-05-13
| | |
| * | eval/typval: Fix numbuf parameter documentationZyX2017-05-13
| | |
| * | functests: Get rid of last redraws due to the “line above” issueZyX2017-05-11
| | |
| * | functests: Remove outdated commentsZyX2017-05-10
| | |
| * | functests: Remove some redraw callsZyX2017-05-10
| | |
| * | functests: Remove all wait()sZyX2017-05-10
| | |
| * | eval: Alter E5050 error message, test thatZyX2017-05-10
| | |
| * | doc: Do not assume something is not supported in GUIZyX2017-05-10
| | |
| * | eval: Refactor get_user_input to support dictionaryZyX2017-05-10
| | |
* | | cmake: Support building without LuaJIT. #6736Florian Walch2017-05-13
| | | | | | | | | | | | | | | | | | | | | | | | Compile `nvim` executable against Lua if PREFER_LUA=ON. As the testing library `nvim-test` requires LuaJIT, it is still compiled against LuaJIT. If LuaJIT is not available, `nvim-test` is not built.
* | | Merge pull request #6704 from bfredl/luaexecBjörn Linse2017-05-13
|\ \ \ | | | | | | | | execute lua directly from the remote API
| * | | api: execute lua directly from the remote apiBjörn Linse2017-05-13
| | | |
| * | | API version bumpBjörn Linse2017-05-12
| | |/ | |/|