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
Add header guards for DYTThrObject.h and Serializable.h #29086
Add header guards for DYTThrObject.h and Serializable.h #29086
Conversation
For Serializable.h, we should change #pragma once, because it gets the filename and creates a include guard based on it. If it relocates it becomes problematic, because it is could be possible to have two files with the same name in two different locations. (to avoid such cases we should rely less on #pragma once)
The code-checks are being triggered in jenkins. |
Part of #15248 |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29086/13976
|
A new Pull Request was created by @oshadura (Oksana Shadura) for master. It involves the following packages: CondFormats/RecoMuonObjects @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @tlampen, @ggovi, @pohsun can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@oshadura is it because of this missing the 5600 is failing ? in other words should we test 5600 along with this by default ? |
@mrodozov yup! it should go together, sorry for late notice! |
The tests are being triggered in jenkins.
|
the 'requires external' label isn't really appropriate here... the external PR requires this PR, but this PR does not require the external PR. |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
hi @ggovi, all - any further comments on this PR? |
seems not - could we move on with this PR? |
@ggovi I'm going to integrate this PR in the next IB. Please let me know if you have objections. |
merge |
For Serializable.h, we should change #pragma once, because it gets the filename and creates a include guard based on it. If it relocates it becomes problematic, because it is could be possible to have two files with the same name in two different locations.
(to avoid such cases we should rely less on #pragma once)
PR description:
This PR is a part of cms-sw/cmsdist#5600
cc: @davidlange6 @vgvassilev