Skip to content

Commit

Permalink
Merge pull request #36977 from perrotta/removeDeadCodeInHltriggerBtau
Browse files Browse the repository at this point in the history
Remove dead assignments and simplify HLTmumutkFilter
  • Loading branch information
cmsbuild authored Feb 16, 2022
2 parents a3446c6 + c13d623 commit fec3825
Showing 1 changed file with 5 additions and 23 deletions.
28 changes: 5 additions & 23 deletions HLTrigger/btau/plugins/HLTmumutkFilter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
#include <cmath>
#include <vector>

#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#include "FWCore/ParameterSet/interface/ConfigurationDescriptions.h"

Expand All @@ -13,11 +12,6 @@
#include "DataFormats/VertexReco/interface/Vertex.h"
#include "DataFormats/VertexReco/interface/VertexFwd.h"

#include "RecoVertex/KalmanVertexFit/interface/KalmanVertexFitter.h"
#include "RecoVertex/VertexPrimitives/interface/TransientVertex.h"
#include "TrackingTools/TransientTrack/interface/TransientTrackBuilder.h"
#include "TrackingTools/Records/interface/TransientTrackRecord.h"

#include "HLTmumutkFilter.h"
#include "TMath.h"

Expand Down Expand Up @@ -177,29 +171,17 @@ bool HLTmumutkFilter::hltFilter(edm::Event& iEvent,
throw cms::Exception("BadLogic") << "HLTmumutkFilterr: ERROR: the vertex must have "
<< " at least two muons by definition." << std::endl;

int iTrkFoundRefs = 0;
bool notYetFoundTrackRefs(true);
for (auto cand = trkcands->begin(); cand != trkcands->end(); cand++) {
reco::TrackRef tkRef = cand->get<reco::TrackRef>();
if (tkRef == vertextkRef1 && iTrkFoundRefs == 0 && !track1Matched) {
tkcand = cand;
iTrkFoundRefs++;
track1Matched = true;
break;
}
if (tkRef == vertextkRef2 && iTrkFoundRefs == 0 && !track2Matched) {
tkcand = cand;
iTrkFoundRefs++;
track2Matched = true;
break;
}
if (tkRef == vertextkRef3 && iTrkFoundRefs == 0 && !track3Matched) {
if ((tkRef == vertextkRef1 && !track1Matched) || (tkRef == vertextkRef2 && !track2Matched) ||
(tkRef == vertextkRef3 && !track3Matched)) {
notYetFoundTrackRefs = false;
tkcand = cand;
iTrkFoundRefs++;
track3Matched = true;
break;
}
}
if (iTrkFoundRefs == 0)
if (notYetFoundTrackRefs)
throw cms::Exception("BadLogic") << "HLTmumutkFilterr: ERROR: the vertex must have "
<< " at least one track by definition." << std::endl;

Expand Down

0 comments on commit fec3825

Please sign in to comment.