Merge pull request #25558 from Dr15Jones/clangMuIsoDepositCopyProducer

Remove inappropriate use of std::move
cms-sw · Dec 26, 2018 · 5cb0f3f · 5cb0f3f
2 parents 2f732fb + e102b13
commit 5cb0f3f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/RecoMuon/MuonIsolationProducers/plugins/MuIsoDepositCopyProducer.cc b/RecoMuon/MuonIsolationProducers/plugins/MuIsoDepositCopyProducer.cc
@@ -34,7 +34,7 @@ MuIsoDepositCopyProducer::MuIsoDepositCopyProducer(const ParameterSet& par) :
 
   for (unsigned int i = 0; i < theDepositNames.size(); ++i){
     std::string alias = theConfig.getParameter<std::string>("@module_label");
-    if (theDepositNames[i] != "") alias += "_" + theDepositNames[i];
+    if (!theDepositNames[i].empty()) alias += "_" + theDepositNames[i];
     produces<reco::IsoDepositMap>(theDepositNames[i]).setBranchAlias(alias);
   }
   for (unsigned int iDep = 0; iDep < theInputTags.size(); ++iDep)
@@ -60,7 +60,7 @@ void MuIsoDepositCopyProducer::produce(Event& event, const EventSetup& eventSetu
     Handle<reco::IsoDepositMap > inDep;
     event.getByToken(theInputTokens[iDep], inDep);
 
-    event.put(std::move(std::make_unique<reco::IsoDepositMap>(*inDep)), theDepositNames[iDep]);
+    event.put(std::make_unique<reco::IsoDepositMap>(*inDep), theDepositNames[iDep]);
   }//! end iDep
 
   LogTrace(metname) <<" END OF EVENT " <<"================================";