Commit message (Collapse) | Author | Age | |
---|---|---|---|
* | options: Silence V542 without using comments | ZyX | 2017-05-20 |
| | |||
* | options: Silence V542 the other way | ZyX | 2017-05-20 |
| | | | Still does not work though. | ||
* | syntax: Silence V782 | ZyX | 2017-05-20 |
| | | | | Just another pointer hack used with hash tables. | ||
* | tui: Fix V547: always true condition | ZyX | 2017-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 condition | ZyX | 2017-05-20 |
| | | | This condition was already checked at the surrounding if() at line 2422. | ||
* | tag: Silence V522: potential null dereference | ZyX | 2017-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 condition | ZyX | 2017-05-20 |
| | |||
* | search: Fix V502: ?: ambiguity | ZyX | 2017-05-20 |
| | |||
* | options: Silence V542: odd casts for .def_val | ZyX | 2017-05-20 |
| | |||
* | memfile: Fix V547: always true condition | ZyX | 2017-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 dereference | ZyX | 2017-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 loop | ZyX | 2017-05-20 |
| | |||
* | eval: Silence V614: potentially uninitialized variable | ZyX | 2017-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 V507 | ZyX | 2017-05-20 |
| | |||
* | eval: Silence V782 | ZyX | 2017-05-20 |
| | |||
* | eval/encode: Silence V595 error | ZyX | 2017-05-20 |
| | |||
* | buffer: Silence V519 | ZyX | 2017-05-20 |
| | |||
* | edit: Silence V595 | ZyX | 2017-05-20 |
| | |||
* | *: Fix all V641 errors | ZyX | 2017-05-20 |
| | |||
* | charset: Fix V695: dead branches | ZyX | 2017-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. Keyes | 2017-05-17 |
| | |||
* | oldtests: Mark test_timer's Test_oneshot as flaky | James McCoy | 2017-05-16 |
| | |||
* | Merge #6741 from justinmk/progpath | Justin M. Keyes | 2017-05-15 |
|\ | |||
| * | env_iter: Learn `delim` parameter. | Justin M. Keyes | 2017-05-15 |
| | | |||
| * | vim_getenv: Use v:progpath instead of os_exepath. | Justin M. Keyes | 2017-05-15 |
| | | |||
| * | startup: v:progpath fallback: path_guess_exepath | Justin M. Keyes | 2017-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. Keyes | 2017-05-15 |
|\ \ | |/ |/| | |||
| * | options: make 'highlight' read-only | Björn Linse | 2017-05-15 |
| | | |||
* | | Merge #6480 from ZyX-I/colored-cmdline'/input-dict | Justin M. Keyes | 2017-05-13 |
|\ \ | |||
| * | | eval/typval: Fix numbuf parameter documentation | ZyX | 2017-05-13 |
| | | | |||
| * | | eval: Alter E5050 error message, test that | ZyX | 2017-05-10 |
| | | | |||
| * | | eval: Refactor get_user_input to support dictionary | ZyX | 2017-05-10 |
| | | | |||
* | | | cmake: Support building without LuaJIT. #6736 | Florian Walch | 2017-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/luaexec | Björn Linse | 2017-05-13 |
|\ \ \ | | | | | | | | | execute lua directly from the remote API | ||
| * | | | api: execute lua directly from the remote api | Björn Linse | 2017-05-13 |
| | |/ | |/| | |||
* | | | Merge pull request #6514 from jamessan/gcc-7-fixes | James McCoy | 2017-05-13 |
|\ \ \ | | | | | | | | | Fix GCC 7 issues | ||
| * | | | oldtests: Fix spelling of testname in s:flaky | James McCoy | 2017-05-12 |
| | | | | |||
| * | | | lint | James McCoy | 2017-05-12 |
| | | | | |||
| * | | | regexp_nfa: Fix invalid fallthrough in character class detection | James McCoy | 2017-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 supported | James McCoy | 2017-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 fallthroughs | James McCoy | 2017-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 failure | James McCoy | 2017-05-12 |
| | | | | |||
| * | | | os_unix: Mark mch_exit as NORETURN | James McCoy | 2017-05-12 |
| | | | | |||
| * | | | func_attr: Support "noreturn" attribute | James McCoy | 2017-05-12 |
| |/ / | |||
* | | | Merge #6731 from ZyX-I/fix-oneline-script-skip | Justin M. Keyes | 2017-05-13 |
|\ \ \ | |/ / |/| | | |||
| * | | ex_getln: Fix :lang code execution when skipping | ZyX | 2017-05-13 |
| |/ | | | | | | | Fixes #6727 | ||
* | | doc (#6719) | Justin M. Keyes | 2017-05-11 |
| | | | | | | Closes #6712 | ||
* | | ui_events: erase internal type HlAttrs (#6718) | Björn Linse | 2017-05-11 |
| | | |||
* | | api: always use prefix FUNC_API, also change NOEVAL to REMOTE_ONLY | Björn Linse | 2017-05-10 |
| | | |||
* | | os/shell.c: temporary solution to not put ctrl chars on the screen grid | Björn Linse | 2017-05-10 |
| | |