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
check the bodies of FunctionDecl's too #4104
Conversation
A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_2_X. check the bodies of FunctionDecl's too It involves the following packages: Utilities/StaticAnalyzers @cmsbuild, @Degano, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests. |
Pull request #4104 was updated. @Dr15Jones, @ktf can you please check and sign again. |
@nclopezo, which commit did you check? |
The tests started at 5:05 AM CET, so it they must have included f3f0eda |
Functions not tied to classes. Don't issues warnings from FunctionChecker since this info is written to a file. Have clang geneerate plist files when running ClassDumper,FunctionDumper and FunctionChecker.
OK I decided to not change the scripts. The change is only to FunctionChecker.cpp so that it does not output warnings but still writes them to the file in $LOCALRT/tmp for use in generating the reports. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). |
Check the bodies of FunctionDecl's too.
No description provided.