ZyX
1fb4282f1c
fileio: Fix PVS/V560: end was already checked
...
See condition at line 3309.
2018-04-15 20:27:41 +03:00
ZyX
11f6411771
file_search: Silence PVS/V614: used buffer is never uninitialized
2018-04-15 20:18:20 +03:00
ZyX
d084628c4b
kvec: Silence PVS/V512: it is not needed to fill the whole array
2018-04-15 20:14:55 +03:00
ZyX
a20261fae7
viml/parser/expressions: Fix PVS/V547: condition checked earlier
...
See previous goto with the same label.
2018-04-15 20:11:21 +03:00
ZyX
bc235bf1f6
viml/parser/expressions: Fix PVS/V547: condition checked earlier
...
See previous goto with the same label.
2018-04-15 20:10:48 +03:00
ZyX
6196738a3b
ex_cmds: Fix PVS/V547: expression was already checked in outer if()
2018-04-15 20:08:32 +03:00
ZyX
17e21eae24
eval: Silence PVS/V547: PVS cannot stand !known_val
expressions
2018-04-15 20:07:07 +03:00
ZyX
092b4d9774
*: Add some PVS comments
2018-04-15 20:07:07 +03:00
ZyX
469ba2fb49
ex_getln: Fix PVS/V547: function is stated to never return NULL
2018-04-15 20:07:07 +03:00
ZyX
c90e9df5b5
ex_getln: Fix PVS/V560: there are no longer NULL returns for OOM
2018-04-15 20:07:07 +03:00
ZyX
cad616c94e
ex_docmd: Fix PVS/V547: condition was already checked in surr. if()
2018-04-15 20:07:07 +03:00
ZyX
d7e4449adb
ex_docmd: Fix PVS/V547: due to short-circuiting name_len is never > 4
2018-04-15 20:07:07 +03:00
ZyX
103be37f42
ex_cmds: Fix PVS/V557: possible array overrun in :ascii
...
Some calculation show that with the current setup there will not be enough bytes
occupied for that, barring the case of malicious translation. Still should be
possible to have array overrun with specially crafted translation.
2018-04-15 20:07:07 +03:00
ZyX
aa5008c1f0
ex_cmds: Simplify do_ascii
...
Specifically apply constants like enc_utf8 (which are constants *now*) and
simplify conditions after that. Also some style changes.
2018-04-15 19:01:58 +03:00
ZyX
db7f80302b
ex_cmds: Remove #ifdef USE_GUI
...
`ag` shows that this identifier (`USE_GUI`) is found in exactly one place (zero
after this commit). So I assume macros is never defined.
2018-04-15 18:52:37 +03:00
ZyX
1df9ac1c03
eval: Fix PVS/V547: skipping is now done using eval0
2018-04-15 18:50:02 +03:00
ZyX
f8d574225b
eval: Silence PVS/V547: E882 may be triggered
...
I failed to deduce why analyzer thinks E882 may not be triggered, though
conditions for triggering it are strange: it would trigger E882 only in the
single case “function returned non-number”. Cases “function thrown exception”,
or “built-in sorter encountered error” will neither yield E882 nor stop
sort()/uniq().
Note though that searching test code revealed that neither E702 nor E882 are not
tested anywhere.
2018-04-15 18:45:12 +03:00
ZyX
6f19b9f4e1
eval: Silence PVS/V614: use of potentially uninitialized pointer
...
It is hard to say whether it actually is uninitialized, need to go deeper into
regex code. Probably analyzer did not go that far as regmatch for sure would not
be initialized up until calling NFA/DFA engine functions, which is to be done by
pointer.
2018-04-10 01:52:18 +03:00
ZyX
4d1b3bf317
eval: Fix PVS/V560: unneded check for name_len
...
It is unsigned, zero length would already cause early return and length 1 is
checked earlier in the same condition.
2018-04-10 01:52:08 +03:00
ZyX
07b67f9eff
eval: Fix PVS/V547: ufunc_T is actually an incomplete type
2018-04-10 01:51:52 +03:00
ZyX
536d9a6168
edit: Fix PVS/V560: remove always true/false checks
2018-04-10 01:12:59 +03:00
ZyX
ef57cd81fe
diff: Fix PVS/V547: remove no longer needed OOM handler
2018-04-10 01:12:38 +03:00
ZyX
5acda03576
eval/decode: Silence PVS/V547: code written for 32-bit integers
2018-04-09 10:29:34 +03:00
ZyX
db29fa8164
charset: Fix PVS/V560: remove condition which was already checked
2018-04-09 10:29:33 +03:00
ZyX
1edb4894dc
charset: Silence PVS/V785: remove switch()
...
No idea why it thinks that pre is constant expression, but switch() may be
removed.
2018-04-09 10:29:32 +03:00
ZyX
f8a44b73f0
event/loop: Silence PVS/V547: condition is false in case of no timeout
2018-04-09 10:29:32 +03:00
ZyX
92759ef34f
buffer: Fix PVS/V560: condition was checked three lines above
2018-04-09 10:29:31 +03:00
ZyX
840027c7f5
buffer: Fix PVS/V547: base is never set to octal
2018-04-09 10:29:30 +03:00
ZyX
93be2ba542
charset: Fix transchar() with multibyte characters
...
It appears that transchar() was working under assumption that
`transchar_nonprint()` may be used for multibyte characters while its
documentation stated exact opposite. It was not actually untrue though, except
that longer buffer would be needed then the one stated in documentation. But it
is false now with assert().
2018-04-09 10:29:29 +03:00
ZyX
58a5699a44
buffer: PVS/V557: Refactor maketitle()
2018-04-09 10:29:28 +03:00
ZyX
2a6491cdf8
os/env: Refactor home_replace() to the new style and to return length
2018-04-09 10:29:26 +03:00
ZyX
0d7daaad98
charset,*: Refactor transstr()
2018-04-09 01:39:33 +03:00
ZyX
e670756676
memory: Fix documentation of some string copying functions
2018-04-09 01:36:52 +03:00
ZyX
414ef75ee6
buffer: Beautify code a bit
2018-04-09 01:35:23 +03:00
ZyX
1fc09b5b98
eval/typval_encode: Silence PVS/V501,V547,V779
...
These kinds of warnings are inevitable for generic macros.
2018-04-02 00:46:18 +03:00
ZyX
5660482406
api/buffer: Fix PVS/V547: use correct border for lnum
...
Should actually be silencing that for the sake of the case when `long` is
actually not 64-bit. But it appears that Vim had already defined maximal line
number. And even declared that exact value invalid, so no need in silencing.
2018-04-02 00:41:00 +03:00
ZyX
bf160dd668
quickfix: Fix PVS/V560: remove duplicate condition
...
Condition was checked in surrounding if().
2018-04-02 00:31:29 +03:00
ZyX
6b84f7813c
pvscheck: Also provide source tree root to plog-converter
2018-04-02 00:16:22 +03:00
ZyX
65d0b8ed32
pvscheck: Also produce fullhtml reports
2018-04-02 00:15:31 +03:00
ZyX
7eceac218e
pvscheck: Make realdir work with nonexistent directories
2018-04-02 00:07:05 +03:00
Björn Linse
98e7112390
msg: do not scroll entire screen ( #8088 )
2018-03-31 11:12:27 +02:00
Marty E. Plummer
362346f563
build/CMake: LANGUAGES config option #8203
...
Signed-off-by: Marty E. Plummer <hanetzer@startmail.com>
2018-03-31 10:59:45 +02:00
Justin M. Keyes
c8fd7ce08b
ci/AppVeyor: cover MinGW and MSVC in PRs ( #8206 )
2018-03-30 21:54:39 +02:00
Justin M. Keyes
30f6ff7021
.gitignore
2018-03-30 17:47:25 +02:00
Utkarsh Maheshwari
f21867a15c
timer_pause: stop the timer resource ( #8199 )
...
If the timer isn't stopped, it still emits events which consume some CPU.
Fix #8188
2018-03-30 17:09:29 +02:00
James McCoy
e54ff10d44
lint
2018-03-29 10:32:12 -04:00
James McCoy
b5a47f4907
Merge pull request #6377 from yagebu/option-fixes
...
options: fixes and some refactoring for number options
Closes #6696
2018-03-29 09:11:22 -04:00
Utkarsh Anand
e9cf40f2b6
build/NetBSD: use kinfo_proc2; undef uint64_t ( #8197 )
...
closes #8196
For historical reasons, uint64_t and friends are defined both as
typedefs and macros. Some platforms that do that define the macros as
identity (#define uint64_t uint64_t), others like NetBSD define to the
backing type (#define uint64_t __uint64_t). This is normally
transparent, except when multiple levels of macro expansions are used
inconsistently.
2018-03-29 10:37:49 +02:00
Justin M. Keyes
bbca3142e8
ci/AppVeyor: build MinGW only on master branch ( #8193 )
...
Else the build takes too long.
2018-03-29 09:17:44 +02:00
James McCoy
79f9c2d9c6
Merge branch 'master' into yagebu/option-fixes
2018-03-28 21:54:39 -04:00