aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | 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
| | |
* | | 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.
* | | startup: init v:progpath before calling vim_getenv (#6755)Justin M. Keyes2017-05-17
| | |
* | | 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
| | | |
* | | | Merge #6480 from ZyX-I/colored-cmdline'/input-dictJustin M. Keyes2017-05-13
|\ \ \ \
| * | | | eval/typval: Fix numbuf parameter documentationZyX2017-05-13
| | | | |
| * | | | eval: Alter E5050 error message, test thatZyX2017-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
| | |/ / | |/| |
* | | | Merge pull request #6514 from jamessan/gcc-7-fixesJames McCoy2017-05-13
|\ \ \ \ | | | | | | | | | | Fix GCC 7 issues
| * | | | oldtests: Fix spelling of testname in s:flakyJames McCoy2017-05-12
| | | | |
| * | | | lintJames McCoy2017-05-12
| | | | |
| * | | | regexp_nfa: Fix invalid fallthrough in character class detectionJames McCoy2017-05-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the end character in a range matches a different standard range (e.g., [0-z]), the range would be incorrectly detected as the class of the end character (CLASS_az). Instead of using a fallthrough, immediately FAIL when the end character doesn't match the expected range.
| * | | | *: Use __attribute__((fallthrough)) where comments aren't supportedJames McCoy2017-05-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although GCC now detects possibly unintentional fall through, there rules around which the comments are detected are rather strict. In cases where a comment isn't detected, upstream [recommends] using their fallthrough attribute. [recommends]: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817#c11
| * | | | *: Comment intentional fallthroughsJames McCoy2017-05-12
| | | | | | | | | | | | | | | | | | | | | | | | | Falling through a switch case should be commented so it's clear that behavior is intentional.
| * | | | ex_cmds: Use NULL, not NUL, to check for strrchr failureJames McCoy2017-05-12
| | | | |
| * | | | os_unix: Mark mch_exit as NORETURNJames McCoy2017-05-12
| | | | |
| * | | | func_attr: Support "noreturn" attributeJames McCoy2017-05-12
| |/ / /
* | | | Merge #6731 from ZyX-I/fix-oneline-script-skipJustin M. Keyes2017-05-13
|\ \ \ \ | |/ / / |/| | |
| * | | ex_getln: Fix :lang code execution when skippingZyX2017-05-13
| |/ / | | | | | | | | | Fixes #6727
* | | doc (#6719)Justin M. Keyes2017-05-11
| | | | | | | | | Closes #6712
* | | ui_events: erase internal type HlAttrs (#6718)Björn Linse2017-05-11
| | |
* | | api: always use prefix FUNC_API, also change NOEVAL to REMOTE_ONLYBjörn Linse2017-05-10
| | |
* | | os/shell.c: temporary solution to not put ctrl chars on the screen gridBjörn Linse2017-05-10
| | |
* | | api: add metadata for ui eventsBjörn Linse2017-05-10
| | |