Skip to content
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

Cppcheck integration #202

Merged
merged 2 commits into from
Oct 16, 2019
Merged

Cppcheck integration #202

merged 2 commits into from
Oct 16, 2019

Conversation

marktwtn
Copy link
Collaborator

The detected and fixed errors list:

  • Assign a value that is never used
  • Reassign a value before the old one has been used
  • The scope of the variable can be reduced
  • The behaviour of using void pointers in calculations is undefined
  • Uninitialized variable
  • Shadow variables

The file cppcheck_suppress suppresses the errors which should not be
treated as errors.
The reason is written in the file.

The sanitizer.mk is renamed as dynamic-analysis.mk.
And there is another file static-analysis.mk for static code analysis.

Related to #173.

@marktwtn marktwtn requested a review from jserv October 16, 2019 10:34
@marktwtn marktwtn self-assigned this Oct 16, 2019
docs/build-n-test.md Outdated Show resolved Hide resolved
The detected and fixed errors list:
- Assign a value that is never used
- Reassign a value before the old one has been used
- The scope of the variable can be reduced
- The behaviour of using void pointers in calculations is undefined
- Uninitialized variable
- Shadow variables

The file cppcheck_suppress suppresses the errors which should not be
treated as errors.
The reason is written in the file.

The sanitizer.mk is renamed as dynamic-analysis.mk.
And there is another file static-analysis.mk for static code analysis.

Related to #173.
@jserv jserv merged commit 118d00f into develop Oct 16, 2019
@marktwtn marktwtn deleted the cppcheck branch October 16, 2019 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants