Merge pull request #47320 from iarspider/gcc14-nodiscard

Fix improper use of std::unique and std::remove

Author: cmsbuild

DQM/HLTEvF/plugins/TrigObjTnPHistColl.cc

@@ -122,7 +122,8 @@ void TrigObjTnPHistColl::FilterSelector::mergeTrigKeys(trigger::Keys& keys,
 
 void TrigObjTnPHistColl::FilterSelector::cleanTrigKeys(trigger::Keys& keys) {
   std::sort(keys.begin(), keys.end());
-  std::unique(keys.begin(), keys.end());
+  auto last = std::unique(keys.begin(), keys.end());
+  keys.erase(last, keys.end());
   while (!keys.empty() && keys.back() == std::numeric_limits<trigger::size_type>::max()) {
     keys.pop_back();
   }

DQMOffline/RecoB/src/BTagDifferentialPlot.cc

@@ -222,7 +222,8 @@ void BTagDifferentialPlot::bookHisto(DQMStore::IBooker& ibook) {
   stream << constVariableValue.second << "_"
          << "_Vs_" << diffVariableName;
   commonName += stream.str();
-  std::remove(commonName.begin(), commonName.end(), ' ');
+  auto last = std::remove(commonName.begin(), commonName.end(), ' ');
+  commonName.erase(last, commonName.end());
   std::replace(commonName.begin(), commonName.end(), '.', 'v');
 
   std::string label(commonName);

DQMOffline/RecoB/src/EtaPtBin.cc

@@ -34,7 +34,8 @@ std::string EtaPtBin::buildDescriptionString(const bool& etaActive_,
 
   std::string descr(stream.str());
   // remove blanks which are introduced when adding doubles
-  std::remove(descr.begin(), descr.end(), ' ');
+  auto last = std::remove(descr.begin(), descr.end(), ' ');
+  descr.erase(last, descr.end());
   std::replace(descr.begin(), descr.end(), '.', 'v');
 
   return descr;

PhysicsTools/PatAlgos/plugins/PATElectronProducer.cc

@@ -724,7 +724,8 @@ void PATElectronProducer::produce(edm::Event& iEvent, const edm::EventSetup& iSe
 
           //remove duplicates
           std::sort(selectedCells.begin(), selectedCells.end());
-          std::unique(selectedCells.begin(), selectedCells.end());
+          auto last = std::unique(selectedCells.begin(), selectedCells.end());
+          selectedCells.erase(last, selectedCells.end());
 
           // Retrieve the corresponding RecHits
 
@@ -968,7 +969,8 @@ void PATElectronProducer::produce(edm::Event& iEvent, const edm::EventSetup& iSe
 
       //remove duplicates
       std::sort(selectedCells.begin(), selectedCells.end());
-      std::unique(selectedCells.begin(), selectedCells.end());
+      auto last = std::unique(selectedCells.begin(), selectedCells.end());
+      selectedCells.erase(last, selectedCells.end());
 
       // Retrieve the corresponding RecHits
 

PhysicsTools/PatAlgos/plugins/PATPhotonProducer.cc

@@ -417,7 +417,8 @@ void PATPhotonProducer::produce(edm::Event& iEvent, const edm::EventSetup& iSetu
 
     //remove duplicates
     std::sort(selectedCells.begin(), selectedCells.end());
-    std::unique(selectedCells.begin(), selectedCells.end());
+    auto last = std::unique(selectedCells.begin(), selectedCells.end());
+    selectedCells.erase(last, selectedCells.end());
 
     // Retrieve the corresponding RecHits
 

RecoEcal/EgammaClusterProducers/src/InterestingDetIdCollectionProducer.cc

@@ -194,7 +194,8 @@ void InterestingDetIdCollectionProducer::produce(edm::StreamID,
 
   //unify the vector
   std::sort(indexToStore.begin(), indexToStore.end());
-  std::unique(indexToStore.begin(), indexToStore.end());
+  auto last = std::unique(indexToStore.begin(), indexToStore.end());
+  indexToStore.erase(last, indexToStore.end());
 
   iEvent.put(std::make_unique<DetIdCollection>(indexToStore), interestingDetIdCollection_);
 }

RecoEcal/EgammaClusterProducers/src/InterestingDetIdFromSuperClusterProducer.cc

@@ -193,7 +193,8 @@ void InterestingDetIdFromSuperClusterProducer::produce(edm::StreamID,
 
   //unify the vector
   std::sort(indexToStore.begin(), indexToStore.end());
-  std::unique(indexToStore.begin(), indexToStore.end());
+  auto last = std::unique(indexToStore.begin(), indexToStore.end());
+  indexToStore.erase(last, indexToStore.end());
 
   auto detIdCollection = std::make_unique<DetIdCollection>(indexToStore);
 

RecoEcal/EgammaClusterProducers/src/ReducedRecHitCollectionProducer.cc

@@ -117,7 +117,8 @@ void ReducedRecHitCollectionProducer::produce(edm::Event& iEvent, const edm::Eve
   }
 
   std::sort(xtalsToStore.begin(), xtalsToStore.end());
-  std::unique(xtalsToStore.begin(), xtalsToStore.end());
+  auto last = std::unique(xtalsToStore.begin(), xtalsToStore.end());
+  xtalsToStore.erase(last, xtalsToStore.end());
 
   //   std::cout << "New Collection " << reducedHitsCollection_ << " size is " << miniRecHitCollection->size() << " original is " << recHitsHandle->size() << std::endl;
   iEvent.put(std::move(miniRecHitCollection), reducedHitsCollection_);

RecoEgamma/EgammaIsolationAlgos/plugins/EgammaIsoESDetIdCollectionProducer.cc

@@ -114,7 +114,8 @@ void EgammaIsoESDetIdCollectionProducer::produce(edm::StreamID, edm::Event& iEve
 
   //unify the vector
   std::sort(indexToStore.begin(), indexToStore.end());
-  std::unique(indexToStore.begin(), indexToStore.end());
+  auto last = std::unique(indexToStore.begin(), indexToStore.end());
+  indexToStore.erase(last, indexToStore.end());
 
   auto detIdCollection = std::make_unique<DetIdCollection>(indexToStore);
 

RecoEgamma/EgammaIsolationAlgos/plugins/EgammaIsoHcalDetIdCollectionProducer.cc

@@ -125,7 +125,8 @@ void EgammaIsoHcalDetIdCollectionProducer::produce(edm::Event& iEvent, const edm
 
   //unify the vector
   std::sort(indexToStore.begin(), indexToStore.end());
-  std::unique(indexToStore.begin(), indexToStore.end());
+  auto last = std::unique(indexToStore.begin(), indexToStore.end());
+  indexToStore.erase(last, indexToStore.end());
 
   auto detIdCollection = std::make_unique<DetIdCollection>(indexToStore);