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
Pass ConsumesCollector to ModifyObjectValueBase-derived classes in the constructor #25617
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25617/7907
|
@cmsbuild, please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages: CommonTools/CandAlgos @perrotta, @cmsbuild, @santocch, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@makortel did you check whether this PR doesn't disrupt any object to be used in FWlite? |
@perrotta As far as I can tell, all products are retrieved with |
The code-checks are being triggered in jenkins. |
d20c019
to
b7b86a1
Compare
Sorry for the noise. I was thinking to do some other migration as well as part of this PR, but a closer look made me realize it is best to leave it to a subsequent PR. The force-push restored the earlier state. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25617/7992
|
+1
|
+1 |
merge |
It was noticed in #25593 (comment) that
ModifyObjectValueBase
-derived classes (likepat::ObjectModifier
) takingConsumesCollector
not via the constructor but viasetConsumes()
function call. This PR restructuresModifyObjectValueBase
hierarchy such that theConsumesCollector
is passed via constructor.Tested in CMSSW_10_5_X_2019-01-04-1100, no changes expected.