cms-sw · smuzaffar · May 20, 2026
diff --git a/CalibCalorimetry/EcalLaserAnalyzer/plugins/EcalTestPulseAnalyzer.cc b/CalibCalorimetry/EcalLaserAnalyzer/plugins/EcalTestPulseAnalyzer.cc
@@ -288,7 +288,6 @@ void EcalTestPulseAnalyzer::analyze(const edm::Event& e, const edm::EventSetup&
   double chi2pn = 0;
   double ypnrange[50];
   double dsum = 0.;
-  double dsum1 = 0.;
   double bl = 0.;
   double val_max = 0.;
   int samplemax = 0;
@@ -435,10 +434,8 @@ void EcalTestPulseAnalyzer::analyze(const edm::Event& e, const edm::EventSetup&
       }
       // Remove pedestal
       //====================
-      for (dsum = 0., dsum1 = 0., k = 0; k < _presample; k++) {
+      for (dsum = 0., k = 0; k < _presample; k++) {
         dsum += adc[k];
-        if (k < _presample - 1)
-          dsum1 += adc[k];
       }
 
       bl = dsum / ((double)_presample);
@@ -561,10 +558,8 @@ void EcalTestPulseAnalyzer::analyze(const edm::Event& e, const edm::EventSetup&
 
       // Remove pedestal
       //====================
-      for (dsum = 0., dsum1 = 0., k = 0; k < _presample; k++) {
+      for (dsum = 0., k = 0; k < _presample; k++) {
         dsum += adc[k];
-        if (k < _presample - 1)
-          dsum1 += adc[k];
       }
 
       bl = dsum / ((double)_presample);

diff --git a/CalibTracker/SiStripChannelGain/plugins/DeDxDiscriminatorLearner.cc b/CalibTracker/SiStripChannelGain/plugins/DeDxDiscriminatorLearner.cc
@@ -124,10 +124,9 @@ void DeDxDiscriminatorLearner::algoAnalyze(const edm::Event& iEvent, const edm::
   edm::Handle<reco::TrackCollection> trackCollectionHandle;
   iEvent.getByToken(m_tracksTag, trackCollectionHandle);
 
-  unsigned track_index = 0;
   for (TrajTrackAssociationCollection::const_iterator it = trajTrackAssociationHandle->begin();
        it != trajTrackAssociationHandle->end();
-       ++it, track_index++) {
+       ++it) {
     const Track& track = *it->val;
     const Trajectory& traj = *it->key;
 

diff --git a/Calibration/HcalCalibAlgos/plugins/AnalyzerMinbias.cc b/Calibration/HcalCalibAlgos/plugins/AnalyzerMinbias.cc
@@ -306,10 +306,16 @@ void AnalyzerMinbias::analyze(const edm::Event& iEvent, const edm::EventSetup& i
     const edm::Handle<L1GlobalTriggerObjectMapRecord> gtObjectMapRecord = iEvent.getHandle(tok_hltL1GtMap_);
     if (gtObjectMapRecord.isValid()) {
       const std::vector<L1GlobalTriggerObjectMap>& objMapVec = gtObjectMapRecord->gtObjectMap();
+#ifdef EDM_ML_DEBUG
       int ii(0);
+#endif
       bool ok(false), fill(true);
       for (std::vector<L1GlobalTriggerObjectMap>::const_iterator itMap = objMapVec.begin(); itMap != objMapVec.end();
+#ifdef EDM_ML_DEBUG
            ++itMap, ++ii) {
+#else
+           ++itMap) {
+#endif
         bool resultGt = (*itMap).algoGtlResult();
         if (resultGt == 1) {
           ok = true;

diff --git a/Calibration/IsolatedParticles/plugins/IsolatedGenParticles.cc b/Calibration/IsolatedParticles/plugins/IsolatedGenParticles.cc
@@ -653,9 +653,8 @@ void IsolatedGenParticles::analyze(const edm::Event &iEvent, const edm::EventSet
       }
     }
 
-    unsigned int indx;
     HepMC::GenEvent::particle_const_iterator p;
-    for (p = myGenEvent->particles_begin(), indx = 0; p != myGenEvent->particles_end(); ++p, ++indx) {
+    for (p = myGenEvent->particles_begin(); p != myGenEvent->particles_end(); ++p) {
       int pdgId = ((*p)->pdg_id());
       int ix = particleCode(pdgId);
       if (ix >= 0) {
@@ -770,9 +769,8 @@ void IsolatedGenParticles::analyze(const edm::Event &iEvent, const edm::EventSet
       }
     }  // loop over gen particles
 
-    unsigned int indx;
     reco::GenParticleCollection::const_iterator p;
-    for (p = genParticles->begin(), indx = 0; p != genParticles->end(); ++p, ++indx) {
+    for (p = genParticles->begin(); p != genParticles->end(); ++p) {
       int pdgId = (p->pdgId());
       int ix = particleCode(pdgId);
       if (ix >= 0) {

diff --git a/Calibration/IsolatedParticles/plugins/StudyCaloGen.cc b/Calibration/IsolatedParticles/plugins/StudyCaloGen.cc
@@ -407,9 +407,8 @@ void StudyCaloGen::analyze(const edm::Event &iEvent, const edm::EventSetup &iSet
       }
     }
 
-    unsigned int indx;
     HepMC::GenEvent::particle_const_iterator p;
-    for (p = myGenEvent->particles_begin(), indx = 0; p != myGenEvent->particles_end(); ++p, ++indx) {
+    for (p = myGenEvent->particles_begin(); p != myGenEvent->particles_end(); ++p) {
       int pdgId = ((*p)->pdg_id());
       int ix = particleCode(pdgId);
       if (ix >= 0) {
@@ -524,9 +523,8 @@ void StudyCaloGen::analyze(const edm::Event &iEvent, const edm::EventSetup &iSet
       }
     }  // loop over gen particles
 
-    unsigned int indx;
     reco::GenParticleCollection::const_iterator p;
-    for (p = genParticles->begin(), indx = 0; p != genParticles->end(); ++p, ++indx) {
+    for (p = genParticles->begin(); p != genParticles->end(); ++p) {
       int pdgId = (p->pdgId());
       int ix = particleCode(pdgId);
       if (ix >= 0) {

diff --git a/Calibration/IsolatedParticles/plugins/StudyCaloResponse.cc b/Calibration/IsolatedParticles/plugins/StudyCaloResponse.cc
@@ -1083,10 +1083,9 @@ int StudyCaloResponse::trackPID(const reco::Track* pTrack,
                                 const edm::Handle<reco::GenParticleCollection>& genParticles) {
   int id(0);
   if (genParticles.isValid()) {
-    unsigned int indx;
     reco::GenParticleCollection::const_iterator p;
     double mindR(999.9);
-    for (p = genParticles->begin(), indx = 0; p != genParticles->end(); ++p, ++indx) {
+    for (p = genParticles->begin(); p != genParticles->end(); ++p) {
       int pdgId = std::abs(p->pdgId());
       int idx = (pdgId == 11) ? 1 : ((pdgId == 211) ? 2 : ((pdgId == 321) ? 3 : ((pdgId == 2212) ? 4 : 0)));
       if (idx > 0) {

diff --git a/DPGAnalysis/Skims/src/TagProbeMassProducer.cc b/DPGAnalysis/Skims/src/TagProbeMassProducer.cc
@@ -82,9 +82,8 @@ void TagProbeMassProducer::produce(edm::Event& iEvent, const edm::EventSetup& iS
       vprobes.push_back(probes->refAt(i));
     }
 
-    int itag = 0;
     edm::RefToBaseVector<reco::Candidate>::const_iterator tag = vtags.begin();
-    for (; tag != vtags.end(); ++tag, ++itag) {
+    for (; tag != vtags.end(); ++tag) {
       int iprobe = 0;
       edm::RefToBaseVector<reco::Candidate>::const_iterator probe = vprobes.begin();
       for (; probe != vprobes.end(); ++probe, ++iprobe) {

diff --git a/DQM/SiStripCommissioningAnalysis/src/CalibrationAlgorithm.cc b/DQM/SiStripCommissioningAnalysis/src/CalibrationAlgorithm.cc
@@ -49,8 +49,7 @@ void CalibrationAlgorithm::extract(const std::vector<TH1*>& histos) {
 
   // Extract histograms
   std::vector<TH1*>::const_iterator ihis = histos.begin();
-  unsigned int cnt = 0;
-  for (; ihis != histos.end(); ihis++, cnt++) {
+  for (; ihis != histos.end(); ihis++) {
     // Check for NULL pointer
     if (!(*ihis)) {
       continue;

diff --git a/DQM/SiStripCommissioningAnalysis/src/CalibrationScanAlgorithm.cc b/DQM/SiStripCommissioningAnalysis/src/CalibrationScanAlgorithm.cc
@@ -51,8 +51,7 @@ void CalibrationScanAlgorithm::extract(const std::vector<TH1*>& histos) {
 
   // Extract histograms
   std::vector<TH1*>::const_iterator ihis = histos.begin();
-  unsigned int cnt = 0;
-  for (; ihis != histos.end(); ihis++, cnt++) {
+  for (; ihis != histos.end(); ihis++) {
     // Check for NULL pointer
     if (!(*ihis)) {
       continue;

diff --git a/DQM/SiStripMonitorHardware/src/SiStripFEDMonitor.cc b/DQM/SiStripMonitorHardware/src/SiStripFEDMonitor.cc
@@ -424,10 +424,8 @@ void SiStripFEDMonitorPlugin::getMajority(const std::vector<std::pair<unsigned i
   unsigned int lCounter = 0;
 
   //std::cout << " --- First element: addr = " << lMajAddr << " counter = " << lCounter << std::endl;
-  unsigned int iele = 0;
   //bool foundMaj = false;
-  for (; lIter != aFeMajVec.end(); ++lIter, ++iele) {
-    //std::cout << " ---- Ele " << iele << " " << (*lIter).first << " " << (*lIter).second << " ref " << lMajAddr << std::endl;
+  for (; lIter != aFeMajVec.end(); ++lIter) {
     if ((*lIter).second == lMajAddr) {
       ++lCounter;
       //std::cout << " ----- =ref: Counter = " << lCounter << std::endl;
@@ -444,7 +442,6 @@ void SiStripFEDMonitorPlugin::getMajority(const std::vector<std::pair<unsigned i
       lCounter = 0;
       lMajAddr = (*lIter).second;
       --lIter;
-      --iele;
     }
   }
   // AV Bug here? The check has to be done regardless foundMaj == false or true

diff --git a/DQMServices/Components/plugins/DQMStoreStats.cc b/DQMServices/Components/plugins/DQMStoreStats.cc
@@ -506,8 +506,7 @@ void DQMStoreStats::calcIgProfDump(Folder& root) {
   root.mainrows_cumulative(sql_statement);
   root.summary(sql_statement);
   VIterator<Folder*> subsystems = root.CreateIterator();
-  size_t ii = 0;
-  for (subsystems.First(); !subsystems.IsDone(); subsystems.Next(), ++ii) {
+  for (subsystems.First(); !subsystems.IsDone(); subsystems.Next()) {
     subsystems.CurrentItem()->mainrows(sql_statement);
     subsystems.CurrentItem()->parents(sql_statement);
     subsystems.CurrentItem()->children(sql_statement);

diff --git a/DataFormats/Common/interface/RefVector.h b/DataFormats/Common/interface/RefVector.h
@@ -180,8 +180,7 @@ namespace edm {
     pointers.reserve(this->size());
     helpers.reserve(this->size());
 
-    size_type key = 0;
-    for (const_iterator i = begin(), e = end(); i != e; ++i, ++key) {
+    for (const_iterator i = begin(), e = end(); i != e; ++i) {
       member_type const* address = i->isNull() ? nullptr : &**i;
       pointers.push_back(address);
       helpers.emplace_back(i->id(), i->key());

diff --git a/EventFilter/CSCTFRawToDigi/plugins/CSCTFAnalyzer.cc b/EventFilter/CSCTFRawToDigi/plugins/CSCTFAnalyzer.cc
@@ -178,8 +178,7 @@ void CSCTFAnalyzer::analyze(const edm::Event &e, const edm::EventSetup &c) {
     edm::Handle<L1CSCTrackCollection> tracks;
     e.getByToken(L1CST_Tok, tracks);
     if (tracks.isValid()) {
-      int nTrk = 0;
-      for (L1CSCTrackCollection::const_iterator trk = tracks->begin(); trk < tracks->end(); trk++, nTrk++) {
+      for (L1CSCTrackCollection::const_iterator trk = tracks->begin(); trk < tracks->end(); trk++) {
         int sector = 6 * (trk->first.endcap() - 1) + trk->first.sector() - 1;
         int tbin = trk->first.BX();
         edm::LogInfo("CSCTFAnalyzer") << "   Track sector: " << (sector + 1) << "  tbin: " << tbin << " "

diff --git a/EventFilter/GctRawToDigi/plugins/L1GctInternJetProducer.cc b/EventFilter/GctRawToDigi/plugins/L1GctInternJetProducer.cc
@@ -60,8 +60,7 @@ void L1GctInternJetProducer::produce(edm::StreamID, edm::Event& iEvent, const ed
     //std::cout << "apparently the collection was found" <<std::endl;
     L1GctInternJetDataCollection::const_iterator jetItr = hwIntJetCands->begin();
     L1GctInternJetDataCollection::const_iterator jetEnd = hwIntJetCands->end();
-    int i;
-    for (i = 0; jetItr != jetEnd; ++jetItr, ++i) {
+    for (; jetItr != jetEnd; ++jetItr) {
       //std::cout << " JetS a plenty" <<std::endl;
       if (!jetItr->empty() && (!centralBxOnly_ || jetItr->bx() == 0)) {
         double et = (jetItr->oflow() ? (double)0xfff : (double)jetItr->et()) * etSumLSB + 1.e-6;

diff --git a/EventFilter/L1TRawToDigi/plugins/implementations_stage1/RCTEmRegionPacker.cc b/EventFilter/L1TRawToDigi/plugins/implementations_stage1/RCTEmRegionPacker.cc
@@ -28,9 +28,8 @@ namespace l1t {
       std::vector<uint32_t> load[36];
 
       for (int i = 0; i <= 0; ++i) {
-        int n = 0;
         PhysicsToBitConverter converter[18];
-        for (auto j = caloregion->begin(); j != caloregion->end(); ++j, ++n) {
+        for (auto j = caloregion->begin(); j != caloregion->end(); ++j) {
           int et = (int)j->et();
           int overFlow = (int)j->overFlow();
           int fineGrain = (int)j->fineGrain();
@@ -58,8 +57,7 @@ namespace l1t {
           }
         }  //end calo region
 
-        int m = 0;
-        for (auto j = caloemcand->begin(); j != caloemcand->end(); ++j, ++m) {
+        for (auto j = caloemcand->begin(); j != caloemcand->end(); ++j) {
           int rank = (int)j->rank();
           int index = (int)j->index();
           int rctCrate = (int)j->rctCrate();

diff --git a/Fireworks/Core/src/CmsShowCommon.cc b/Fireworks/Core/src/CmsShowCommon.cc
@@ -137,10 +137,9 @@ void CmsShowCommon::permuteColors() {
 void CmsShowCommon::randomizeColors() {
   //   printf("Doing random_shuffle on existing colors ...\n");
 
-  int vi = 0;
   for (FWEventItemsManager::const_iterator i = m_context->eventItemsManager()->begin();
        i != m_context->eventItemsManager()->end();
-       ++i, ++vi) {
+       ++i) {
     FWDisplayProperties prop = (*i)->defaultDisplayProperties();
 
     int col = rand() % 17;  // randomize in first row of palette

diff --git a/Fireworks/Core/src/FWModelChangeManager.cc b/Fireworks/Core/src/FWModelChangeManager.cc
@@ -85,9 +85,8 @@ void FWModelChangeManager::endChanges() {
   //makes sure that 'changeSignalsAreDone is called if changeSignalsAreComing_ is sent
   bool guard(false);
   if (0 == --m_depth) {
-    unsigned int index = 0;
     for (std::set<const FWEventItem*>::iterator itChanges = m_itemChanges.begin(); itChanges != m_itemChanges.end();
-         ++itChanges, ++index) {
+         ++itChanges) {
       if (!guard) {
         // std::shared_ptr<FWModelChangeManager> done(this, &sendChangeSignalsAreDone);
         guard = true;
@@ -109,7 +108,7 @@ void FWModelChangeManager::endChanges() {
     }
     m_itemChanges.clear();
 
-    for (size_t ci = 0, ce = m_changes.size(), si = 0; ci != ce; ++ci, ++si) {
+    for (size_t ci = 0, ce = m_changes.size(); ci != ce; ++ci) {
       FWModelIds& changes = m_changes[ci];
       if (not changes.empty()) {
         if (!guard) {

diff --git a/Fireworks/ParticleFlow/plugins/FWPFCandidateWithHitsProxyBuilder.cc b/Fireworks/ParticleFlow/plugins/FWPFCandidateWithHitsProxyBuilder.cc
@@ -75,10 +75,9 @@ void FWPFCandidateWithHitsProxyBuilder::build(const FWEventItem* iItem,
   if (candidates == nullptr)
     return;
 
-  Int_t idx = 0;
   initPFRecHitsCollections();
   for (reco::PFCandidateCollection::const_iterator it = candidates->begin(), itEnd = candidates->end(); it != itEnd;
-       ++it, ++idx) {
+       ++it) {
     TEveCompound* comp = createCompound();
     setupAddElement(comp, product);
     // printf("products size %d/%d \n", (int)iItem->size(), product->NumChildren());

diff --git a/Geometry/HGCalCommonData/test/HGCalCellUVTester.cc b/Geometry/HGCalCommonData/test/HGCalCellUVTester.cc
@@ -126,9 +126,8 @@ void HGCalCellUVTester::beginRun(edm::Run const &iRun, edm::EventSetup const &iS
   edm::LogVerbatim("HGCalGeom") << "\nHGCalCellUVTester:: nCells " << nCells << " and placement index between "
                                 << indexMin << " and " << indexMax << "\n\n";
   auto start_t = std::chrono::high_resolution_clock::now();
-  unsigned int kk(0);
   std::unordered_map<int32_t, HGCalParameters::waferInfo>::const_iterator itr = hgpar_->waferInfoMap_.begin();
-  for (; itr != hgpar_->waferInfoMap_.end(); ++itr, ++kk) {
+  for (; itr != hgpar_->waferInfoMap_.end(); ++itr) {
     //for (auto itr = hgpar_->waferInfoMap_.begin(); itr != hgpar_->waferInfoMap_.end(); ++itr){
     if ((itr->second).part != HGCalTypes::WaferFull) {
       int indx = itr->first;

diff --git a/Geometry/HGCalGeometry/test/HGCalGeometryCheck.cc b/Geometry/HGCalGeometry/test/HGCalGeometryCheck.cc
@@ -103,8 +103,7 @@ void HGCalGeometryCheck::beginRun(const edm::Run&, const edm::EventSetup& iSetup
     sprintf(name, "RZ_%s", nameDetectors_[ih].c_str());
     sprintf(title, "R vs Z for %s", nameDetectors_[ih].c_str());
     h_RZ_.emplace_back(fs->make<TH2D>(name, title, nbinZ_, zmin_, zmax_, nbinR_, rmin_, rmax_));
-    unsigned int k(0);
-    for (int lay = layerF; lay <= layerL; ++lay, ++k) {
+    for (int lay = layerF; lay <= layerL; ++lay) {
       sprintf(name, "Mod_%s_L%d", nameDetectors_[ih].c_str(), lay);
       sprintf(title, "Modules in layer %d in %s", lay, nameDetectors_[ih].c_str());
       h_Mod_.emplace_back(fs->make<TH1D>(name, title, 200, -50, 50));

diff --git a/Geometry/HcalTowerAlgo/test/HcalGeometryDump.cc b/Geometry/HcalTowerAlgo/test/HcalGeometryDump.cc
@@ -60,8 +60,7 @@ void HcalGeometryDump::analyze(const edm::Event& /*iEvent*/, const edm::EventSet
     }
     edm::LogVerbatim("HCalGeom") << detIds.size() << " valid Ids for subdetector " << subdet;
     std::sort(detIds.begin(), detIds.end());
-    int counter = 0;
-    for (std::vector<unsigned int>::const_iterator i = detIds.begin(); i != detIds.end(); ++i, ++counter) {
+    for (std::vector<unsigned int>::const_iterator i = detIds.begin(); i != detIds.end(); ++i) {
       HcalDetId hid = HcalDetId(*i);
       auto cell = caloGeom->getGeometry(*i);
       edm::LogVerbatim("HCalGeom") << hid << "\tCaloCellGeometry " << cell->getPosition() << "\tHcalGeometry "

diff --git a/HLTrigger/Muon/plugins/HLTL1MuonSelector.cc b/HLTrigger/Muon/plugins/HLTL1MuonSelector.cc
@@ -61,9 +61,8 @@ void HLTL1MuonSelector::produce(edm::StreamID, edm::Event& iEvent, const edm::Ev
   LogTrace(metname) << "Number of muons " << muColl->size() << endl;
 
   L1MuonParticleCollection::const_iterator it;
-  L1MuonParticleRef::key_type l1ParticleIndex = 0;
 
-  for (it = muColl->begin(); it != muColl->end(); ++it, ++l1ParticleIndex) {
+  for (it = muColl->begin(); it != muColl->end(); ++it) {
     const L1MuGMTExtendedCand muonCand = (*it).gmtMuonCand();
     unsigned int quality = 0;
     bool valid_charge = false;

diff --git a/L1Trigger/CSCTrackFinder/plugins/CSCTFanalyzer.cc b/L1Trigger/CSCTrackFinder/plugins/CSCTFanalyzer.cc
@@ -84,10 +84,8 @@ void CSCTFanalyzer::analyze(edm::Event const& e, edm::EventSetup const& es) {
 
     for (CSCCorrelatedLCTDigiCollection::DigiRangeIterator csc = LCTs.product()->begin(); csc != LCTs.product()->end();
          csc++) {
-      int lctId = 0;
-
       CSCCorrelatedLCTDigiCollection::Range range1 = LCTs.product()->get((*csc).first);
-      for (CSCCorrelatedLCTDigiCollection::const_iterator lct = range1.first; lct != range1.second; lct++, lctId++) {
+      for (CSCCorrelatedLCTDigiCollection::const_iterator lct = range1.first; lct != range1.second; lct++) {
         int station = (*csc).first.station() - 1;
         int cscId = (*csc).first.triggerCscId() - 1;
         int sector = (*csc).first.triggerSector() - 1 + ((*csc).first.endcap() == 1 ? 0 : 6);

diff --git a/L1Trigger/L1ExtraFromDigis/src/L1ExtraParticlesProd.cc b/L1Trigger/L1ExtraFromDigis/src/L1ExtraParticlesProd.cc
@@ -165,8 +165,8 @@ void L1ExtraParticlesProd::produce(edm::Event &iEvent, const edm::EventSetup &iS
       //       cout << "HW muons" << endl ;
       vector<L1MuGMTExtendedCand>::const_iterator muItr = hwMuCands.begin();
       vector<L1MuGMTExtendedCand>::const_iterator muEnd = hwMuCands.end();
-      for (int i = 0; muItr != muEnd; ++muItr, ++i) {
-        // 	 cout << "#" << i
+      for (; muItr != muEnd; ++muItr) {
+        // 	 cout << "#"
         // 	      << " name " << muItr->name()
         // 	      << " empty " << muItr->empty()
         // 	      << " pt " << muItr->ptIndex()

diff --git a/L1Trigger/L1TCalorimeter/plugins/L1TStage2Layer1Producer.cc b/L1Trigger/L1TCalorimeter/plugins/L1TStage2Layer1Producer.cc
@@ -171,8 +171,7 @@ void L1TStage2Layer1Producer::produce(edm::Event& iEvent, const edm::EventSetup&
 
     // loop over ECAL TPs
     EcalTrigPrimDigiCollection::const_iterator ecalItr;
-    int nEcal = 0;
-    for (ecalItr = ecalTPs->begin(); ecalItr != ecalTPs->end(); ++ecalItr, ++nEcal) {
+    for (ecalItr = ecalTPs->begin(); ecalItr != ecalTPs->end(); ++ecalItr) {
       int ieta = ecalItr->id().ieta();
       int iphi = ecalItr->id().iphi();
 
@@ -197,8 +196,7 @@ void L1TStage2Layer1Producer::produce(edm::Event& iEvent, const edm::EventSetup&
 
     // loop over HCAL TPs
     HcalTrigPrimDigiCollection::const_iterator hcalItr;
-    int nHcal = 0;
-    for (hcalItr = hcalTPs->begin(); hcalItr != hcalTPs->end(); ++hcalItr, ++nHcal) {
+    for (hcalItr = hcalTPs->begin(); hcalItr != hcalTPs->end(); ++hcalItr) {
       int ieta = hcalItr->id().ieta();
       int iphi = hcalItr->id().iphi();
       int ver = hcalItr->id().version();

diff --git a/L1Trigger/L1TCommon/plugins/L1TMuonLegacyConverter.cc b/L1Trigger/L1TCommon/plugins/L1TMuonLegacyConverter.cc
@@ -135,7 +135,7 @@ void L1TMuonLegacyConverter::produce(edm::Event& iEvent, const edm::EventSetup&
 
       vector<L1MuGMTExtendedCand>::const_iterator muItr = simMuCands.begin();
       vector<L1MuGMTExtendedCand>::const_iterator muEnd = simMuCands.end();
-      for (int i = 0; muItr != muEnd; ++muItr, ++i) {
+      for (; muItr != muEnd; ++muItr) {
         if (!muItr->empty()) {
           // keep x and y components non-zero and protect against roundoff.
           double pt = muPtScale->getPtScale()->getLowEdge(muItr->ptIndex()) + 1.e-6;

diff --git a/L1Trigger/L1THGCal/src/backend/HGCalMulticlusteringImpl.cc b/L1Trigger/L1THGCal/src/backend/HGCalMulticlusteringImpl.cc
@@ -74,9 +74,8 @@ void HGCalMulticlusteringImpl::clusterizeDR(const std::vector<edm::Ptr<l1t::HGCa
                                             const HGCalTriggerGeometryBase& triggerGeometry) {
   std::vector<l1t::HGCalMulticluster> multiclustersTmp;
 
-  int iclu = 0;
   for (std::vector<edm::Ptr<l1t::HGCalCluster>>::const_iterator clu = clustersPtrs.begin(); clu != clustersPtrs.end();
-       ++clu, ++iclu) {
+       ++clu) {
     double minDist = dr_;
     int targetMulticlu = -1;
 

diff --git a/PhysicsTools/PatExamples/bin/PatBasicFWLiteJetUnitTest.cc b/PhysicsTools/PatExamples/bin/PatBasicFWLiteJetUnitTest.cc
@@ -55,13 +55,12 @@ int main(int argc, char* argv[]) {
   // ----------------------------------------------------------------------
 
   // loop the events
-  unsigned int iEvent = 0;
   fwlite::Event ev(inFile);
   TStopwatch timer;
   timer.Start();
 
   unsigned int nEventsAnalyzed = 0;
-  for (ev.toBegin(); !ev.atEnd(); ++ev, ++iEvent) {
+  for (ev.toBegin(); !ev.atEnd(); ++ev) {
     edm::EventBase const& event = ev;
 
     // Handle to the jet collection