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
CLANG warning cleanup #3614
CLANG warning cleanup #3614
Conversation
Cleanup some CLANG compiler warning in Core packages related to unused variables and expressions.
A new Pull Request was created by @wddgit (W. David Dagenhart) for CMSSW_7_1_X. CLANG warning cleanup It involves the following packages: DataFormats/FWLite @cmsbuild, @Degano, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks. |
@@ -238,7 +238,7 @@ void testRefInROOT::testRefFirst() | |||
for(edmtest::OtherThingCollection::const_iterator itOther=pOthers->begin(), itEnd=pOthers->end() ; | |||
itOther != itEnd; ++itOther) { | |||
//std::cout <<"getting ref"<<std::endl; | |||
itOther->ref.get()->a; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not a fan of printing stuff from unit tests. How about testing that it isn't some large value?
Instead of eliminating the unused warning by printing the unused expression, compare it to an arbitrary large number.
+1 |
Cleanup some CLANG compiler warning in Core
packages related to unused variables and expressions.