Multi thread porting

Function storeBadEvents removed because it asks for the get and book of a ME in step1.
Hi @deguio <https://github.com/deguio>,
I was asking @rovere <https://github.com/rovere> how to deal with it and he
suggested me to do so. I think the reason is that we want only one instance
of the ibooker and the igetter, but I might be wrong.
Passing the ibooker and the igetter to the functions would mean to switch,
e.g., from

void PFClient::createResolutionPlots(std::string& folder, std::string& name)
to
void PFClient::createResolutionPlots(DQMStore::IBooker& ibooker,
std::string& folder, std::string& name)
?

All the other comments have been addressed.

Thank you,
Leonardo

On 2 August 2014 10:58, deguio <notifications@github.com> wrote:

> In DQMOffline/PFTau/plugins/PFClient.h:
>
> > @@ -40,7 +39,8 @@ class PFClient: public edm::EDAnalyzer {
> >    bool efficiencyFlag_;
> >    bool profileFlag_;
> >
> > -  DQMStore* dqmStore_;
> > +  DQMStore::IBooker * ibooker_;
>
> I would prefer if you could pass the ibooker and the igetter to the
> functions as you did for DQMStep1 instead of storing their pointers. the
> code as it is shouldn't give problems though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-sw/cmssw/pull/4849/files#r15728143>.
>