diff options
| author | dundargoc <gocdundar@gmail.com> | 2023-10-24 17:44:02 +0200 |
|---|---|---|
| committer | dundargoc <33953936+dundargoc@users.noreply.github.com> | 2024-04-17 13:12:37 +0200 |
| commit | b0f9228179bf781eec76d1aaf346b56a7e64cd5d (patch) | |
| tree | df694fa87bd62a0dfd5d594a145b20677330007a | |
| parent | 96d0c709b66e5c690800dc2d88c723b01f5f0561 (diff) | |
| download | rneovim-b0f9228179bf781eec76d1aaf346b56a7e64cd5d.tar.gz rneovim-b0f9228179bf781eec76d1aaf346b56a7e64cd5d.tar.bz2 rneovim-b0f9228179bf781eec76d1aaf346b56a7e64cd5d.zip | |
docs: remove mentions of scan-build
These same checks are already checked by the clang-analyzer job, so this
duplication is not necessary.
| -rw-r--r-- | CONTRIBUTING.md | 17 | ||||
| -rw-r--r-- | README.md | 1 | ||||
| -rw-r--r-- | src/nvim/CMakeLists.txt | 10 |
3 files changed, 8 insertions, 20 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b10289a694..88d2a99029 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -8,7 +8,7 @@ If you want to help but don't know where to start, here are some low-risk/isolated tasks: - Try a [complexity:low] issue. -- Fix bugs found by [Clang](#clang-scan-build) or [Coverity](#coverity). +- Fix bugs found by [Coverity](#coverity). - [Merge a Vim patch] (requires strong familiarity with Vim) - NOTE: read the above link before sending improvements to "runtime files" (anything in `runtime/`). - Vimscript and documentation files are (mostly) maintained by [Vim](https://github.com/vim/vim), not Nvim. @@ -132,21 +132,6 @@ Each pull request must pass the automated builds on [Cirrus CI] and [GitHub Acti - To see CI results faster in your PR, you can temporarily set `TEST_FILE` in [test.yml](https://github.com/neovim/neovim/blob/e35b9020b16985eee26e942f9a3f6b045bc3809b/.github/workflows/test.yml#L29). -### Clang scan-build - -View the [Clang report] to see potential bugs found by the Clang -[scan-build](https://clang-analyzer.llvm.org/scan-build.html) analyzer. - -- Search the Neovim commit history to find examples: - ```bash - git log --oneline --no-merges --grep clang - ``` -- To verify a fix locally, run `scan-build` like this: - ```bash - rm -rf build/ - scan-build --use-analyzer=/usr/bin/clang make - ``` - ### Coverity Coverity runs against the master build. To view the defects you must @@ -6,7 +6,6 @@ </h1> [](https://scan.coverity.com/projects/2227) -[](https://neovim.io/doc/reports/clang) [](https://repology.org/metapackage/neovim) [](https://buildd.debian.org/neovim) [](https://github.com/neovim/neovim/releases/) diff --git a/src/nvim/CMakeLists.txt b/src/nvim/CMakeLists.txt index 45ad738255..64bd40f5f0 100644 --- a/src/nvim/CMakeLists.txt +++ b/src/nvim/CMakeLists.txt @@ -833,9 +833,12 @@ add_glob_target( FLAGS --quiet EXCLUDE ${EXCLUDE_CLANG_TIDY}) -# These are the same warnings as https://neovim.io/doc/reports/clang/. The -# checks we ignore are meant to be removed eventually, but we can only do so -# after we properly fix the problems without breaking CI. +# The checks we ignore are meant to be removed eventually, but we can only +# enable each warning after we fix all instances of that specific warning as to +# not break CI. +if(APPLE) + string(APPEND CLANG_ANALYZER_IGNORE "-clang-analyzer-core.NonNullParamChecker,") +endif() add_glob_target( TARGET clang-analyzer COMMAND ${CLANG_TIDY_PRG} @@ -849,6 +852,7 @@ add_glob_target( -clang-analyzer-core.uninitialized.Assign, -clang-analyzer-optin.performance.Padding, -clang-analyzer-security.insecureAPI.strcpy, + ${CLANG_ANALYZER_IGNORE} ' EXCLUDE ${EXCLUDE_CLANG_TIDY}) |