mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-29 04:50:03 +01:00
40 lines
1.3 KiB
Markdown
40 lines
1.3 KiB
Markdown
# About
|
|
|
|
This is a cppcheck wrapper script with appropriate parameters for
|
|
checking RIOT.
|
|
It accepts a branch name as an argument which is used to limit the
|
|
scope of the check. Other parameters will be passed to cppcheck, so
|
|
you can further modify its behavior.
|
|
|
|
# Example usage
|
|
|
|
Check all files changed in the current branch against the branch named
|
|
'master':
|
|
|
|
./dist/tools/cppcheck/check.sh master
|
|
|
|
Check all files but ignore warnings about unused struct members:
|
|
|
|
./dist/tools/cppcheck/check.sh --suppress=unassignedVariable
|
|
|
|
Check all files changed in the current branch against the branch named
|
|
'master', ignoring warnings about unassigned variables:
|
|
|
|
./dist/tools/cppcheck/check.sh master --suppress=unassignedVariable
|
|
|
|
# Default options
|
|
|
|
This script suppresses warnings of the type "unusedStructMember" by default. If
|
|
you want to get warnings about "unusedStructMembers" run the script with the
|
|
--show-unused-struct option:
|
|
./dist/tools/cppcheck/check.sh --show-unused-struct [BRANCH] [options to be passed]
|
|
|
|
# What to do about the findings
|
|
|
|
You should read the code carefully. While cppcheck certainly produces
|
|
valuable information, it can also warn about code that is actually OK.
|
|
If this happens, you can add an "inline suppression" like this:
|
|
|
|
/* cppcheck-suppress passedByValue */
|
|
timex_t timex_add(const timex_t a, const timex_t b);
|