aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
|
* option: Fix PVS/V560: EXPAND_BOOL_SETTINGS already caused early returnZyX2018-04-16
|
* option: Fix PVS/V547: allocator never returns NULL nowZyX2018-04-16
|
* option: Fix PVS/V517: duplicate check for titlelenZyX2018-04-16
|
* option: Silence PVS/V594: GLOBAL_WO macros is not really necessaryZyX2018-04-16
| | | | In that case it is completely possible to just use w_allbuf_opt directly.
* ops: Fix PVS/V560: pre may not be zero as that was already checkedZyX2018-04-16
|
* ops: Fix PVS/V614: use of uninitialized variableZyX2018-04-15
| | | | I have actually no idea how code managed to work and not trigger ASAN/etc here. It does not look like a false positive at all.
* ops: Fix PVS/V547: allocator never returns NULLZyX2018-04-15
|
* ops: Fix PVS/V547: did_change is almost always setZyX2018-04-15
| | | Only exception is early `goto theend`.
* ops: Silence PVS/V614: regtype is always initializedZyX2018-04-15
|
* message: Fix PVS/V547: c is never equal to KS_ZEROZyX2018-04-15
| | | | | | | | | | | | | Since `c` there is a result of evaluating `TO_SPECIAL` macros it may be only one of the following three things: 1. K_SPECIAL 2. K_ZERO (note: not KS_ZERO) 3. negative integer resulting from evaluating TERMCAP2KEY macro. All variants here are negative and thus fail next !IS_SPECIAL(c) check (negative is special). If `c` was really NUL it would fall into the `!IS_SPECIAL(c)` block and use whatever character is third in `<80>{a}{b}` combo. For `<Nul>` it is X (`<80><ff>X`).
* memline: Fix PVS/V547: allocator now never returns NULLZyX2018-04-15
|
* memline: Fix PVS/V560: allocator now never returns NULLZyX2018-04-15
|
* mbyte: Fix PVS/V557: do not do useless jobZyX2018-04-15
| | | | I do not see how array overrun is actually possible, but still EUC encodings may do fine without a cycle.
* if_cscope: Fix PVS/V560: condition would result in earlier returnZyX2018-04-15
| | | | Previous block just checks for totmatches being zero and returns if it is. And totmatches is unsigned, so `totmatches > 0` may never be true.
* os/fs: Fix PVS/V560: condition was already checked in while()ZyX2018-04-15
| | | | It is not possible to enter while loop body with unsigned2 == 0 if loop condition requires unsigned1 < unsigned2.
* fileio: Silence PVS/V614: buffer is initialized if conv_restlen > 0ZyX2018-04-15
|
* fileio: Fix PVS/V547: split is not usedZyX2018-04-15
| | | This is removed functionality, specifically no longer supported 16-bit integers.
* fileio: Fix PVS/V560: end was already checkedZyX2018-04-15
| | | See condition at line 3309.
* file_search: Silence PVS/V614: used buffer is never uninitializedZyX2018-04-15
|
* kvec: Silence PVS/V512: it is not needed to fill the whole arrayZyX2018-04-15
|
* viml/parser/expressions: Fix PVS/V547: condition checked earlierZyX2018-04-15
| | | See previous goto with the same label.
* viml/parser/expressions: Fix PVS/V547: condition checked earlierZyX2018-04-15
| | | See previous goto with the same label.
* ex_cmds: Fix PVS/V547: expression was already checked in outer if()ZyX2018-04-15
|
* eval: Silence PVS/V547: PVS cannot stand `!known_val` expressionsZyX2018-04-15
|
* *: Add some PVS commentsZyX2018-04-15
|
* ex_getln: Fix PVS/V547: function is stated to never return NULLZyX2018-04-15
|
generated by cgit (git 2.34.1) at 2025-06-01 11:20:21 +0000