diff --git a/DQMOffline/JetMET/interface/JetAnalyzer_HeavyIons.h b/DQMOffline/JetMET/interface/JetAnalyzer_HeavyIons.h index 228a17ccdd51b..a0b4a67c4493c 100644 --- a/DQMOffline/JetMET/interface/JetAnalyzer_HeavyIons.h +++ b/DQMOffline/JetMET/interface/JetAnalyzer_HeavyIons.h @@ -85,7 +85,7 @@ class JetAnalyzer_HeavyIons : public DQMEDAnalyzer { public: explicit JetAnalyzer_HeavyIons (const edm::ParameterSet&); - virtual ~JetAnalyzer_HeavyIons(); + ~JetAnalyzer_HeavyIons() override; void analyze(const edm::Event&, const edm::EventSetup&) override; void bookHistograms(DQMStore::IBooker &, edm::Run const &, edm::EventSetup const &) override; diff --git a/DQMOffline/JetMET/src/JetAnalyzer_HeavyIons.cc b/DQMOffline/JetMET/src/JetAnalyzer_HeavyIons.cc index 51fdc0b792f1d..38d4db4b473d3 100644 --- a/DQMOffline/JetMET/src/JetAnalyzer_HeavyIons.cc +++ b/DQMOffline/JetMET/src/JetAnalyzer_HeavyIons.cc @@ -61,115 +61,115 @@ JetAnalyzer_HeavyIons::JetAnalyzer_HeavyIons(const edm::ParameterSet& iConfig) : // need to initialize the PF cand histograms : which are also event variables if(isPFJet){ - mNPFpart = 0; - mPFPt = 0; - mPFEta = 0; - mPFPhi = 0; - - mSumPFPt = 0; - mSumPFPt_eta = 0; - mSumSquaredPFPt = 0; - mSumSquaredPFPt_eta = 0; - mSumPFPt_HF = 0; + mNPFpart = nullptr; + mPFPt = nullptr; + mPFEta = nullptr; + mPFPhi = nullptr; + + mSumPFPt = nullptr; + mSumPFPt_eta = nullptr; + mSumSquaredPFPt = nullptr; + mSumSquaredPFPt_eta = nullptr; + mSumPFPt_HF = nullptr; - mPFDeltaR = 0; - mPFDeltaR_Scaled_R = 0; + mPFDeltaR = nullptr; + mPFDeltaR_Scaled_R = nullptr; for(int ieta=0; ietasize(); iCScand++){ - assert(csCandidateColl->size() < pfCandidateColl->size()); + assert(csCandidateColl->size() <= pfCandidateColl->size()); const reco::PFCandidate csCandidate = csCandidateColl->at(iCScand); const reco::PFCandidate pfCandidate = pfCandidateColl->at(iCScand); int ieta=0; @@ -832,10 +832,8 @@ void JetAnalyzer_HeavyIons::analyze(const edm::Event& mEvent, const edm::EventSe } } - if (mNJets_40) - mNJets_40->Fill(nJet_40); - - } // end recojet loop 222 + } + if (mNJets_40) mNJets_40->Fill(nJet_40); numbers.clear();