aboutsummaryrefslogtreecommitdiff
path: root/scripts/pvscheck.sh
Commit message (Collapse)AuthorAge
* build: remove PVSdundargoc2023-11-12
| | | | | | | We already have an extensive suite of static analysis tools we use, which causes a fair bit of redundancy as we get duplicate warnings. PVS is also prone to give false warnings which creates a lot of work to identify and disable.
* PVS fixesdundargoc2023-11-10
| | | | | | | | | | | | | * build(PVS): exclude mpack and klib as they are external dependencies * build(PVS): suppress warning V601 See https://pvs-studio.com/en/docs/warnings/v601/ * fix(PVS/V009): add top-level message * fix(PVS/V547): expression 'p != NULL' is always true * fix(PVS/V547): expression '* termpp == NULL' is always false
* build(PVS): exclude build directorydundargoc2023-10-07
| | | | This is to prevent reports on generated files.
* docs: fix typosdundargoc2022-11-02
|
* refactor: fix clang and PVS warnings (#19569)zeertzjq2022-07-29
| | | | The last commit didn't actually disable V1028 because of a typo. Fix the typo so it is actually disabled.
* build(pvscheck): disable PVS/V1028 (#19553)zeertzjq2022-07-29
| | | | | Most casts where PVS warns for V1028 aren't added to prevent overflows in the first place, but to avoid other warnings, like printf argument or -Wconversion warnings. PVS/V1028 is more annoying than useful.
* fix/PVS #17863dundargoc2022-04-24
| | | | | | | | | | | | | | | | | | * fix(PVS/V002): disable rule completely V002: "Some diagnostic messages may contain incorrect line number in this file." This particular check seems unreliable. It says on their website https://pvs-studio.com/en/docs/warnings/v002/ that this warning occurs when there are multiline pragmas, but there are none in extmark.c. * fix(PVS/V756): ignore "counter is not used inside a nested loop" warning The nested loop starts with "AutoCmd *ac = ap->cmds" so "ap" is definitely used. * fix(PVS/V560): disable "a part of conditional expression is always true" * fix(PVS/V614): potentially uninitialized variable 'blen' used
* ci: exclude lua-cjson from PVS report #15859dundargoc2021-10-05
|
* fix(pvs): Exclude xdiff from analysisJustin M. Keyes2021-09-24
| | | | | ref 088161a9459a ref fbe88ef8f5a4
* fix(PVS): disable "typo" warnings V1051, V1074Justin M. Keyes2021-09-24
| | | | | | | | | | | | | | | | | | | PVS is worried about typos. Now we need it to stop worrying... Disable these checks entirely, they are all false positives. tui.c:1873 V1074 Boundary between escape sequence and string is unclear. The escape sequence ends with a letter and the next character is also a letter. Check for typos. tui.c:1983 V1074 Boundary between escape sequence and string is unclear. The escape sequence ends with a letter and the next character is also a letter. Check for typos. regexp_nfa.c:6189 V1051 Consider checking for misprints. It's possible that the 'pim->result' should be checked here. screen.c:2928 V1051 Consider checking for misprints. It's possible that the 'vcol_sbr' should be checked here. screen.c:3187 V1051 Consider checking for misprints. It's possible that the 'line_attr' should be checked here. screen.c:3267 V1051 Consider checking for misprints. It's possible that the 'multi_attr' should be checked here. screen.c:4747 V1051 Consider checking for misprints. It's possible that the 'redraw_next' should be checked here. syntax.c:3448 V1051 Consider checking for misprints. It's possible that the 'arg_end' should be checked here. syntax.c:3625 V1051 Consider checking for misprints. It's possible that the 'arg_end' should be checked here. tui.c:1836 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here. tui.c:1863 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here. tui.c:1882 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here.
* chore: PVS/V1042 - ignore warning globally.Dundar Göc2021-07-17
| | | | | V1042 is a warning that a file has a copyleft license, which is an irrelevant warning to open-source projects.
* fix(pvscheck): Switch URL to pvs-studio.comJames McCoy2021-04-29
| | | | [skip ci]
* pvs: Exclude xdiff from analysis and comment mungingJames McCoy2020-11-10
|
* Use the free PVS-Studio licenseJames McCoy2020-11-10
| | | | | As of release 7.10, PVS Studio requires a license, even for the "check me" comment based analyzing.
* pvscheck.sh: Remove --verbose flag #10473Ihor Antonov2019-07-12
| | | | | pvscheck.sh gives error: Flag could not be matched: verbose
* pvscheck.sh: set --sourcetree-root [ci skip]Justin M. Keyes2019-01-19
| | | | | | | | | | | | | | | Reverts previous experiment. PVS root is working correctly, one can observe this in the PVS-studio.err file, for example: /usr/local/clang-7.0.0/lib/clang/7.0.0/include/stddef.h:51:1: warning: V677 ... /usr/local/clang-7.0.0/lib/clang/7.0.0/include/stddef.h:132:1: warning: V677 ... ./src/nvim/fileio.c:1382:1: warning: V1026 ... ./src/nvim/fileio.c:1388:1: warning: V1026 ... The "./src/nvim/…" paths are correctly rooted, yet PVS somehow still thinks it should analyze "/usr/local/clang-7.0.0/…". See also: https://stackoverflow.com/q/44906903
* pvscheck.sh: do not set --sourcetree-root [ci skip]Justin M. Keyes2019-01-19
| | | | | System headers should be ignored by PVS, but somehow aren't. See also: https://stackoverflow.com/q/44906903
* pvscheck.sh: ignore stddef.hJustin M. Keyes2019-01-18
| | | | | This system header should be ignored by PVS. See also: https://stackoverflow.com/q/44906903
* pvscheck.sh: Fix download URL #9500Justin M. Keyes2019-01-14
| | | | | - old URL redirects to https://www.viva64.com/en/pvs-studio-download/ - page now contains tgz files for macOS and Linux; the pattern must match only the Linux URL
* pvscheck.sh: Skip install if dir existsJustin M. Keyes2019-01-08
|
* pvscheck: Disable V011 warningZyX2018-04-22
|
* 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.
* 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
|
* pvscheck: Ignore exit code of pvs-studio-analyzerJames McCoy2018-03-11
| | | | | Since its typically non-zero, the script immediately exits instead of converting the binary log into useful formats.
* pvscheck.sh: auto-detect URL by defaultJustin M. Keyes2017-11-21
| | | | | | | The hardcoded URL breaks very often, this confuses people. Instead, auto-detect if no URL is provided. Also auto-detect if the script is invoked with no arguments.
* pvscheck: Add --environment-cc switchZyX2017-07-04
| | | | | | To be used to make bot-ci able to use clang-4.0 without hacks. [ci skip]
* scripts/pvscheck.sh: fix function renameJustin M. Keyes2017-06-25
|
* scripts/pvscheck.sh: HACK: de-parallelize on CIJustin M. Keyes2017-06-25
| | | | https://github.com/neovim/bot-ci/pull/105#issuecomment-309282132
* scripts/pvscheck.sh: HACK: de-parallelize on CIJustin M. Keyes2017-06-25
| | | | References https://github.com/neovim/bot-ci/pull/105
* scripts/pvscheck.sh: fixesDaniel Hahler2017-06-17
| | | | [ci skip]
* pvscheck: Fix argument handling in do_recheckJames McCoy2017-06-09
| | | | [ci skip]
* pvscheck: Provide arguments to `patch_sources` in correct orderZyX2017-05-20
|
* pvscheck: Handle invalid option error gracefullyZyX2017-05-20
|
* pvscheck: Add --only-analyse modeZyX2017-05-20
|
* pvscheck: Use absolute path for finding test-include.cZyX2017-05-02
| | | | | | It was currently unknown in which directory create_compile_commands will end up in. [ci skip]
* pvscheck: When using --recheck rerun buildZyX2017-05-02
| | | | [ci skip]
* pvscheck: Add --deps to build with all dependenciesZyX2017-05-02
| | | | [ci skip]
* pvscheck: Add --pvs-install modeZyX2017-05-02
| | | | [ci skip]
* pvscheck: Refactor script so that it only `cd`s in a subshellZyX2017-05-02
| | | | [ci skip]
* pvscheck: Create getopts_long implementationZyX2017-05-02
| | | | | | | Needed as argument list is growing large and I absolutely do not find short options provided by getopts being particularly readable for unfamiliar commands. [ci skip]
* pvscheck: Add --pvs key to pvscheck.shZyX2017-05-02
| | | | [ci skip]
* pvscheck: Update pvs-studio URLZyX2017-05-02
| | | | [ci skip]
* pvscheck: Do not use `test x`ZyX2017-05-02
| | | | [ci skip]
* scripts: Add newline after the commentZyX2017-04-19
|
* scripts: Allow patching only build filesZyX2017-04-19
|
* scripts: Do not patch already patched sources in patch modeZyX2017-04-19
| | | Also do not patch header files, that is not needed.
* scripts: Make pvs do not patch separate directory, add patch modeZyX2017-04-19
|
generated by cgit (git 2.34.1) at 2025-04-28 18:48:52 +0000