From 5223dc733a412f6b28b9498643eff4958eb789bf Mon Sep 17 00:00:00 2001
From: Andrea <perrotta@cern.ch>
Date: Sun, 19 Jun 2022 14:30:35 +0200
Subject: [PATCH] Remove duplicate initializations

---
 DQM/HcalTasks/plugins/RecHitTask.cc | 7 +++----
 DQM/HcalTasks/plugins/TPTask.cc     | 2 --
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/DQM/HcalTasks/plugins/RecHitTask.cc b/DQM/HcalTasks/plugins/RecHitTask.cc
index a5bfcc3315199..5f8fef15924fa 100644
--- a/DQM/HcalTasks/plugins/RecHitTask.cc
+++ b/DQM/HcalTasks/plugins/RecHitTask.cc
@@ -450,7 +450,6 @@ RecHitTask::RecHitTask(edm::ParameterSet const& ps)
 
   //  To fill histograms outside of the loop, you need to determine if there were
   //  any valid det ids first
-  uint32_t rawidValid = 0;
   uint32_t rawidHBValid = 0;
   uint32_t rawidHEValid = 0;
 
@@ -479,7 +478,6 @@ RecHitTask::RecHitTask(edm::ParameterSet const& ps)
     //{meUnknownIds1LS->Fill(1); _unknownIdsPresent=true;continue;}
 
     HcalElectronicsId const& eid(rawid);
-    rawidValid = did.rawId();
     if (did.subdet() == HcalBarrel)
       rawidHBValid = did.rawId();
     else if (did.subdet() == HcalEndcap)
@@ -580,8 +578,9 @@ RecHitTask::RecHitTask(edm::ParameterSet const& ps)
     //	^^^ONLINE ONLY!
   }
 
-  //	reset
-  rawidValid = 0;
+  //  To fill histograms outside of the loop, you need to determine if there were
+  //  any valid det ids first
+  uint32_t rawidValid = 0;
 
   int nChsHO = 0;
   int nChsHOCut = 0;
diff --git a/DQM/HcalTasks/plugins/TPTask.cc b/DQM/HcalTasks/plugins/TPTask.cc
index e52d4330d640c..50d417780b0b5 100644
--- a/DQM/HcalTasks/plugins/TPTask.cc
+++ b/DQM/HcalTasks/plugins/TPTask.cc
@@ -939,8 +939,6 @@ TPTask::TPTask(edm::ParameterSet const& ps)
 
   numHBHE = 0;
   numHF = 0;
-  numCutHBHE = 0;
-  numCutHF = 0;
   numMsnHBHE = 0;
   numMsnHF = 0;
   numCutHBHE = 0;