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 fixes for RecoEgamma/ElectronIdentification #5301
Clang fixes for RecoEgamma/ElectronIdentification #5301
Conversation
The clang compiler was warning about member data which were never used.
The clang compiler issued a warning that the declaration of result(...) hid the version of that function from the base class. As the new result(...) was being used and therefore was intended to be different from the base class, issuing a 'using' declaration for the base class function now silences the warning.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_2_X. Clang fixes for RecoEgamma/ElectronIdentification It involves the following packages: RecoEgamma/ElectronIdentification @cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
+1 |
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). @nclopezo can you please take care of it? |
…ntification Clang fixes for RecoEgamma/ElectronIdentification
Fixed clang compiler warnings.