aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
|
* ex_getln: Fix PVS/V560: there are no longer NULL returns for OOMZyX2018-04-15
|
* ex_docmd: Fix PVS/V547: condition was already checked in surr. if()ZyX2018-04-15
|
* ex_docmd: Fix PVS/V547: due to short-circuiting name_len is never > 4ZyX2018-04-15
|
* ex_cmds: Fix PVS/V557: possible array overrun in :asciiZyX2018-04-15
| | | | | | 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.
* ex_cmds: Simplify do_asciiZyX2018-04-15
| | | | Specifically apply constants like enc_utf8 (which are constants *now*) and simplify conditions after that. Also some style changes.
* ex_cmds: Remove #ifdef USE_GUIZyX2018-04-15
| | | | `ag` shows that this identifier (`USE_GUI`) is found in exactly one place (zero after this commit). So I assume macros is never defined.
* eval: Fix PVS/V547: skipping is now done using eval0ZyX2018-04-15
|
* eval: Silence PVS/V547: E882 may be triggeredZyX2018-04-15
| | | | | | | | | | 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.
* eval: Silence PVS/V614: use of potentially uninitialized pointerZyX2018-04-10
| | | | | | 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.
* eval: Fix PVS/V560: unneded check for name_lenZyX2018-04-10
| | | | It is unsigned, zero length would already cause early return and length 1 is checked earlier in the same condition.
* eval: Fix PVS/V547: ufunc_T is actually an incomplete typeZyX2018-04-10
|
* edit: Fix PVS/V560: remove always true/false checksZyX2018-04-10
|
* diff: Fix PVS/V547: remove no longer needed OOM handlerZyX2018-04-10
|
* eval/decode: Silence PVS/V547: code written for 32-bit integersZyX2018-04-09
|
* charset: Fix PVS/V560: remove condition which was already checkedZyX2018-04-09
|
* charset: Silence PVS/V785: remove switch()ZyX2018-04-09
| | | | No idea why it thinks that pre is constant expression, but switch() may be removed.
* event/loop: Silence PVS/V547: condition is false in case of no timeoutZyX2018-04-09
|
* buffer: Fix PVS/V560: condition was checked three lines aboveZyX2018-04-09
|
* buffer: Fix PVS/V547: base is never set to octalZyX2018-04-09
|
* charset: Fix transchar() with multibyte charactersZyX2018-04-09
| | | | | | | 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().
* buffer: PVS/V557: Refactor maketitle()ZyX2018-04-09
|
* os/env: Refactor home_replace() to the new style and to return lengthZyX2018-04-09
|
* charset,*: Refactor transstr()ZyX2018-04-09
|
* memory: Fix documentation of some string copying functionsZyX2018-04-09
|
* buffer: Beautify code a bitZyX2018-04-09
|
* eval/typval_encode: Silence PVS/V501,V547,V779ZyX2018-04-02
| | | These kinds of warnings are inevitable for generic macros.
* api/buffer: Fix PVS/V547: use correct border for lnumZyX2018-04-02
| | | | | 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.
* quickfix: Fix PVS/V560: remove duplicate conditionZyX2018-04-02
| | | Condition was checked in surrounding if().
* pvscheck: Also provide source tree root to plog-converterZyX2018-04-02
|
* pvscheck: Also produce fullhtml reportsZyX2018-04-02
|
* pvscheck: Make realdir work with nonexistent directoriesZyX2018-04-02
|
* msg: do not scroll entire screen (#8088)Björn Linse2018-03-31
|
* build/CMake: LANGUAGES config option #8203Marty E. Plummer2018-03-31
| | | | Signed-off-by: Marty E. Plummer <hanetzer@startmail.com>
* ci/AppVeyor: cover MinGW and MSVC in PRs (#8206)Justin M. Keyes2018-03-30
|
* .gitignoreJustin M. Keyes2018-03-30
|
* timer_pause: stop the timer resource (#8199)Utkarsh Maheshwari2018-03-30
| | | | | If the timer isn't stopped, it still emits events which consume some CPU. Fix #8188
* lintJames McCoy2018-03-29
|
* Merge pull request #6377 from yagebu/option-fixesJames McCoy2018-03-29
|\ | | | | | | | | options: fixes and some refactoring for number options Closes #6696
| * Merge branch 'master' into yagebu/option-fixesJames McCoy2018-03-28
| |\
| * | update testsJakob Schnitzer2017-06-28
| | |
| * | Merge branch 'master' into option-fixesJakob Schnitzer2017-06-28
| |\ \
| * \ \ Merge branch 'master' into option-fixesJakob Schnitzer2017-04-24
| |\ \ \
| * | | | also test set_optionJakob Schnitzer2017-04-01
| | | | |
| * | | | update for changes in master; fix 'window'; testsJakob Schnitzer2017-03-31
| | | | |
| * | | | options: more tests; check first set later; stricter validationJakob Schnitzer2017-03-30
| | | | |
| * | | | options: fix setglobal for buf-local number optionsJakob Schnitzer2017-03-30
| | | | |
| * | | | options: group num_option validation by typeJakob Schnitzer2017-03-30
| | | | |
generated by cgit (git 2.34.1) at 2025-06-24 11:20:10 +0000