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
Made pat::MET const thread-safe #38457
Conversation
Removed use of mutable member in held helper class.
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38457/30668
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages:
@jpata, @cmsbuild, @clacaputo can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fe2183/25675/summary.html Comparison SummarySummary:
|
@@ -289,15 +289,7 @@ | |||
<class name="pat::MET::PackedMETUncertainty" ClassVersion="11"> | |||
<version ClassVersion="11" checksum="3523936012"/> | |||
<version ClassVersion="10" checksum="1984780659"/> | |||
<field name="dpx_" transient="true" /> |
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.
Just for my understanding, removing these fields (up to line 300) doesn't break back compatibility? What does transient="true"
supposed to do? Thanks
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.
Just for my understanding, removing these fields (up to line 300) doesn't break back compatibility?
I believe that to be correct.
What does transient="true" supposed to do?
It tells ROOT not to store those variables. This is why dropping them from the class doesn't affect read back of old files.
+reconstruction
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Removed use of mutable member in held helper class.
This was found by the static analyzer.
PR validation:
Code compiles.