-
Notifications
You must be signed in to change notification settings - Fork 467
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
Code hardening #2335
Code hardening #2335
Conversation
0556a3c
to
df7830b
Compare
df7830b
to
ead7513
Compare
Build frequencies for coverity are restrictive - https://scan.coverity.com/faq#frequency We discussed in #2325 and in Slack that as a result we mostly use it in addition to my exisiting pre-release sanity check. I think for now there's a |
Ran |
Nice one!
|
Stoked to see static analysis incorporated! It has saved me from shooting myself in my foot many, many times. Especially after refactoring stuff. |
Kicked off another coverity scan. I'll tag 3.5.0.beta.3 pending the results of coverity and my release testing. |
Addresses +80% of the issues found by coverity.
I don't seem to be able to "restart" the checking though.