aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* test/unit/undo_spec.lua: fixup after rebase #4985Justin M. Keyes2018-04-27
|
* test/util: move general functions into global helpersJustin M. Keyes2018-04-27
|
* test/unit: some unit tests for undo.c #4985Christopher Waldon2018-04-27
|
* Merge #8218 'Fix errors reported by PVS'Justin M. Keyes2018-04-27
|\ | | | | closes #4983
| * pvscheck: Disable V011 warningZyX2018-04-22
| |
| * spellfile: Fix clint errorZyX2018-04-22
| |
| * syntax: Fix PVS/V560: condition was already checkedZyX2018-04-22
| |
| * spellfile: Fix PVS/V547: allocator now never returns NULLZyX2018-04-22
| |
| * functests: Fix testlint errorsZyX2018-04-22
| |
| * *: Fix clint errorsZyX2018-04-22
| |
| * window: Fix PVS/V547: expression already checked in previous if()ZyX2018-04-22
| |
| * window: Silence PVS/V547: height may be changed by curwin ptrZyX2018-04-22
| |
| * api/vim: Fix PVS/V547: node was already dereferenced, so can’t be NULLZyX2018-04-22
| |
| * undo: Silence PVS/V547: expression may be false on other systemZyX2018-04-22
| |
| * ui_bridge: Silence PVS/V547: assuming stopped may be changed via ptrZyX2018-04-22
| |
| * eval/typval: Silence PVS/V576: format is correctZyX2018-04-22
| |
| * ugrid: Silence PVS/V625: UGRID_FOREACH_CELL may be used for one rowZyX2018-04-22
| |
| * tui: Silence PVS/V560: condition is set to true in prev-previous if()ZyX2018-04-22
| |
| * syntax: Fix PVS/V560: condition was checked in previous if(), breakingZyX2018-04-22
| |
| * syntax: Fix PVS/V547: condition was checkedZyX2018-04-22
| | | | | | | | | | In surrounding if() `off` was checked for being non-zero and in previous if() it was checked for being positive.
| * syntax: Silence PVS/V560: condition is always true at that pointZyX2018-04-22
| |
| * syntax: Silence PVS/V522: retval is always non-NULL on this roundZyX2018-04-22
| |
| * syntax: Fix PVS/V560: condition checked in previous if()ZyX2018-04-22
| |
| * strings: Fix PVS/V547: condition already checked by surrounding ifZyX2018-04-22
| |
| * strings: Fix PVS/V779: wrong assert() positionZyX2018-04-22
| |
| * spellfile: Silence PVS/V547: expression may be true on other systemsZyX2018-04-22
| |
| * spellfile: Fix PVS/V547: allocator never returns NULL nowZyX2018-04-22
| |
| * spellfile: Fix PVS/V560: allocator never returns NULL nowZyX2018-04-22
| |
| * spell: Fix PVS/V512: use of sprintf()ZyX2018-04-22
| |
| * search: Fix PVS/V547: allocator never returns NULL nowZyX2018-04-22
| |
| * search: Fix PVS/V560: condition checked in previous if with gotoZyX2018-04-22
| |
| * search: Fix PVS/V547: condition already checked in surrounding ifZyX2018-04-22
| |
| * search: Fix PVS/V547: checking variable right after assigning itZyX2018-04-22
| |
| * screen: Fix PVS/V560: col never reassigned in this part of codeZyX2018-04-22
| |
| * screen: Fix PVS/V519: duplicate successive assignmentZyX2018-04-22
| |
| * screen: Fix PVS/V560: condition was checked in surrounding ifZyX2018-04-22
| |
| * globals: Do not use `false` for enc_dbcsZyX2018-04-22
| | | | | | | | Based on screen.c code this value is not a boolean, so `false` is not appropriate. It should be either 0 or one of DBCS_… values.
| * screen: Get rid of ScreenLines2 as it is no longer usedZyX2018-04-22
| | | | | | Should also fix some PVS warnings in process.
| * regexp_nfa: Silence PVS/V560: current code is more obviousZyX2018-04-22
| | | | | | | | | | While warning is technically true (cannot enter this case with `state->c` greater then NFA_Z…9) it makes condition less readable and relies on knownledge of enum internal structure.
| * quickfix: Silence PVS/V560: memory is referenced and may be changedZyX2018-04-22
| |
| * quickfix: Fix PVS/V547: allocator never returns NULL nowZyX2018-04-22
| |
| * quickfix: Fix PVS/V547,V560: no conversion with fixed &encodingZyX2018-04-22
| |
| * eval/typval: Remove struct dictitem_S which is not used anywhereZyX2018-04-20
| |
| * event/process: Silence PVS/V547: assuming stream->num_bytes changesZyX2018-04-17
| | | | | | | | Not familiar with the code, but I assume that loop_poll_events can actually change stream->num_bytes, so condition is not always false.
| * pvscheck: Add --update switchZyX2018-04-17
| |
| * pvscheck: Remove outputs before running plog-converterZyX2018-04-17
| | | | | | | | | | | | | | plog-converter behaviour is not the best one when creating fullhtml report and directory already exists: it puts report inside an existing directory. Not sure what exactly it does if inside exists as well, but if I am not mistaking report will not be created.
| * path: Silence PVS/V557: impossible to reach with NUL-terminated stringZyX2018-04-17
| |
| * path: Fix PVS/V560: condition is essentially `A && A`ZyX2018-04-17
| |
| * option: Silence PVS/V547: expression is true on 64-bit systemsZyX2018-04-16
| |
| * option: Fix PVS/V547: var is always NULL at that pointZyX2018-04-16
| |
generated by cgit (git 2.34.1) at 2025-06-24 15:18:12 +0000