-
Notifications
You must be signed in to change notification settings - Fork 6
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
show some Cppcheck messages on file-level #60
base: master
Are you sure you want to change the base?
Conversation
53d68b7
to
9365b78
Compare
This is a bit too much for 1.6.0 - we need to get the |
I have a patch for Cppcheck ready to make I will also hide all these additions in this PR behind the |
I re-wrote the I will be dogfooding this for a few days before I move it out of the draft status. Please also have a look at the documentation changes if they are understandable. |
Seems to be working as expected. Need to go over the documentation changes one more time and then it should be good to go. |
Cppcheck will now show all the individual |
Workarounds for the messages in question will be outlined in an upcoming PR which fleshes out the
Known Issues
section in the documentation.These should probably be shown as warnings but I chose to keep the severity provided by Cppcheck intact.