Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Move contents of beginJob() to endJob() in APVValidationPlots #28942

Merged
merged 1 commit into from
Feb 13, 2020
Merged

Move contents of beginJob() to endJob() in APVValidationPlots #28942

merged 1 commit into from
Feb 13, 2020
+37 −60

Commits on Feb 12, 2020

  1. Move contents of beginJob() to endJob() 

    This move avoids storing TH1* pointers from MonitorElements as member
variables (even though that should not cause any problems because no
work is done during the event loop). Anyway, now it is crystal clear
that this module accesses DQMStore only in endJob(), and does not need
anything from elsewhere in CMSSW.
    @makortel
    makortel committed Feb 12, 2020
    Configuration menu
    Copy the full SHA
    d204878 View commit details
    Browse the repository at this point in the history