Merge pull request #41521 from smuzaffar/db-dqm-llvm16-warn3

[DB-DQM] [CLANG] Fix warnings reported by llvm16 in CLANG IBs
cms-sw · May 9, 2023 · 120a7ed · 120a7ed
2 parents 845df5a + c70506c
commit 120a7ed
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/DQM/BeamMonitor/plugins/FakeBeamMonitor.cc b/DQM/BeamMonitor/plugins/FakeBeamMonitor.cc
@@ -726,16 +726,13 @@ void FakeBeamMonitor::analyze(const Event& iEvent, const EventSetup& iSetup) {
   //  edm::Handle<reco::VertexCollection> PVCollection;
 
   //  if (iEvent.getByToken(pvSrc_, PVCollection)) {
-  int nPVcount = 0;
   int nPVcount_ST = 0;  //For Single Trigger(hence ST)
 
   //    for (reco::VertexCollection::const_iterator pv = PVCollection->begin(); pv != PVCollection->end(); ++pv) {
   for (int tmp_idx = 0; tmp_idx < 10; tmp_idx++) {
     //--- vertex selection
     //    if (pv->isFake() || pv->tracksSize() == 0)
     //      continue;
-    nPVcount++;  // count non fake pv:
-
     //if (JetTrigPass)
     nPVcount_ST++;  //non-fake pv with a specific trigger
 
@@ -761,8 +758,6 @@ void FakeBeamMonitor::analyze(const Event& iEvent, const EventSetup& iSetup) {
 
   }  //loop over pvs
 
-  //    h_nVtx->Fill(nPVcount * 1.);  //no need to change it for average BS
-
   mapNPV[countLumi_].push_back((nPVcount_ST));
 
   //    if (!StartAverage_) {
@@ -853,7 +848,6 @@ void FakeBeamMonitor::FitAndFill(const LuminosityBlock& lumiSeg, int& lastlumi,
   }
 
   int MaxPVs = 0;
-  int countEvtLastNLS_ = 0;
 
   std::map<int, std::vector<int> >::iterator mnpv = mapNPV.begin();
   std::map<int, std::vector<float> >::iterator mpv2 = mapPVy.begin();
@@ -876,7 +870,6 @@ void FakeBeamMonitor::FitAndFill(const LuminosityBlock& lumiSeg, int& lastlumi,
     for (std::vector<int>::iterator mnpvs = (mnpv->second).begin(); mnpvs != (mnpv->second).end(); ++mnpvs) {
       if ((*mnpvs > 0) && (resetHistoFlag_))
         h_nVtx_st->Fill((*mnpvs) * (1.0));
-      countEvtLastNLS_++;
       if ((*mnpvs) > MaxPVs)
         MaxPVs = (*mnpvs);
     }  //loop over second of mapNPV
@@ -895,7 +888,6 @@ void FakeBeamMonitor::FitAndFill(const LuminosityBlock& lumiSeg, int& lastlumi,
   DipPVInfo_.push_back(rndm_->Gaus(10., 5.));      // Rms PV
   DipPVInfo_.push_back(rndm_->Gaus(5., 3.));       // Rms PV err
   DipPVInfo_.push_back(rndm_->Gaus(100., 10.));    // Max PVs
-  countEvtLastNLS_ = 0;
 
   if (onlineMode_) {  // filling LS gap
     // FIXME: need to add protection for the case if the gap is at the resetting LS!