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
Fix all [-Wheader-guard] warnings #13515
Conversation
The patch fixes 138 warnings of the following format: warning: '<..>' is used as a header guard here, followed by #define of a different macro [-Wheader-guard] Tested with Clang 3.8.0. Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
A new Pull Request was created by @davidlt for CMSSW_8_1_X. It involves the following packages: Calibration/EcalAlCaRecoProducers @diguida, @lveldere, @ianna, @cerminar, @Martin-Grunewald, @covarell, @bendavid, @vanbesien, @govoni, @perrotta, @civanch, @cmsbuild, @fwyzard, @davidlange6, @vciulli, @smuzaffar, @Dr15Jones, @cvuosalo, @mdhildreth, @deguio, @slava77, @ggovi, @vadler, @mmusich, @mulhearn, @ssekmen, @thuer, @monttj, @alja, @franzoni can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
Test are already triggered. |
+1 |
+1 |
+1 |
+1 |
Fix all [-Wheader-guard] warnings
The patch fixes 138 warnings of the following format:
Tested with Clang 3.8.0.
Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch