-
Notifications
You must be signed in to change notification settings - Fork 379
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[analyzer] Cppcheck support #3680
Merged
Merged
Commits on Aug 9, 2022
-
This commit adds support for the Cppcheck analyzer. The following two commits are also squashed into this: * [test] Add new test cases for Cppcheck support * [analyzer] Check version compatibility This commit is the resurrection of works made by Márton Csordás. The original pull request Ericsson#2290 had three more refactoring related change sets which were omitted in this. Some changes were added in compared to the original commits: * The Cppcheck report converter is used to get the results in the result_handler. * The hash calculation logic has been uplifted to the latest one. * Some small small compatibility changes were added because of the changes in the code base. Co-authored-by: bruntib <bruntib@users.noreply.github.com> Co-authored-by: vodorok <vodorok@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 67f76f1 - Browse repository at this point
Copy the full SHA 67f76f1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 69715da - Browse repository at this point
Copy the full SHA 69715daView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7d76f26 - Browse repository at this point
Copy the full SHA 7d76f26View commit details -
[WIP-feat] Extend Cppcheck functionality
This patch is a work in progress, there are debug prints all over the place inside Codechecker, treat it accordingly. Will be amended, and split up, when I am back from vacation. Cppcheck now works in a raw directory instead of directly into the workspace folder. Exponential explosion of reports in plist files are now fixed. Checker disable now works on native levels. The `--disable <cppcheck_check>` now properly translates to `--suppress <cppcheck_check> in the cppcheck invocation. The current run configuration of cppcheck is `--enable=all`. Added two new input paramaters: With `cppcheck-addons` extra cppcheck checkers can be specified. With `cppcheck-libraries` the cppcheck library definitions can be added. Cppcheck reports have to be "fixed" for Codechecker to be able to properly interpret them. The actual error message must be added to any multistep report as a last bug path event. Cppcheck checkers are prefixed with the `cppcheck-` prefix.
Configuration menu - View commit details
-
Copy full SHA for 70351a4 - Browse repository at this point
Copy the full SHA 70351a4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7b65929 - Browse repository at this point
Copy the full SHA 7b65929View commit details -
Configuration menu - View commit details
-
Copy full SHA for 111d14e - Browse repository at this point
Copy the full SHA 111d14eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5269926 - Browse repository at this point
Copy the full SHA 5269926View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4aa8f55 - Browse repository at this point
Copy the full SHA 4aa8f55View commit details -
Configuration menu - View commit details
-
Copy full SHA for da08372 - Browse repository at this point
Copy the full SHA da08372View commit details -
Configuration menu - View commit details
-
Copy full SHA for c160769 - Browse repository at this point
Copy the full SHA c160769View commit details -
Configuration menu - View commit details
-
Copy full SHA for c187603 - Browse repository at this point
Copy the full SHA c187603View commit details -
Configuration menu - View commit details
-
Copy full SHA for b90d771 - Browse repository at this point
Copy the full SHA b90d771View commit details
Commits on Aug 10, 2022
-
Newer version of Cppcheck may include some kind of an ID in the output plist filename, so the renaming/moving logic needed to be refactored to be able to work with new Cppcheck releases.
Configuration menu - View commit details
-
Copy full SHA for 7638a41 - Browse repository at this point
Copy the full SHA 7638a41View commit details -
Configuration menu - View commit details
-
Copy full SHA for d87d5b7 - Browse repository at this point
Copy the full SHA d87d5b7View commit details -
Codechecker will incorrectly use the reports folder path, during the creation of reports, when the source directory is not supplied.
Configuration menu - View commit details
-
Copy full SHA for cf64688 - Browse repository at this point
Copy the full SHA cf64688View commit details -
Configuration menu - View commit details
-
Copy full SHA for 106b685 - Browse repository at this point
Copy the full SHA 106b685View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4925ea6 - Browse repository at this point
Copy the full SHA 4925ea6View commit details -
Configuration menu - View commit details
-
Copy full SHA for f9c9fd1 - Browse repository at this point
Copy the full SHA f9c9fd1View commit details -
Configuration menu - View commit details
-
Copy full SHA for a0650a6 - Browse repository at this point
Copy the full SHA a0650a6View commit details
Commits on Aug 14, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 258833e - Browse repository at this point
Copy the full SHA 258833eView commit details
Commits on Aug 19, 2022
-
Configuration menu - View commit details
-
Copy full SHA for d589856 - Browse repository at this point
Copy the full SHA d589856View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.