Skip to content

Commit 678ba28

Browse files
committed
Second round of comments from BenjaminRS
1 parent 239503c commit 678ba28

File tree

3 files changed

+6
-11
lines changed

3 files changed

+6
-11
lines changed

L1Trigger/DTTriggerPhase2/interface/ShowerBuilder.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,11 +126,10 @@ class ShowerBuilder {
126126
public:
127127
// Constructors and destructor
128128
ShowerBuilder(const edm::ParameterSet& pset, edm::ConsumesCollector& iC);
129-
virtual ~ShowerBuilder();
130129

131130
// Main methods
132-
virtual void initialise(const edm::EventSetup& iEventSetup);
133-
virtual void run(edm::Event& iEvent,
131+
void initialise(const edm::EventSetup& iEventSetup);
132+
void run(edm::Event& iEvent,
134133
const edm::EventSetup& iEventSetup,
135134
const DTDigiCollection& digis,
136135
ShowerCandidatePtr& showerCandidate_SL1,

L1Trigger/DTTriggerPhase2/src/MPThetaMatching.cc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
#include "FWCore/MessageLogger/interface/MessageLogger.h"
33
#include "L1Trigger/DTTriggerPhase2/interface/constants.h"
44

5-
using namespace edm;
65
using namespace cmsdt;
76

87
namespace {
@@ -38,7 +37,7 @@ namespace {
3837
// ============================================================================
3938
// Constructors and destructor
4039
// ============================================================================
41-
MPThetaMatching::MPThetaMatching(const ParameterSet &pset)
40+
MPThetaMatching::MPThetaMatching(const edm::ParameterSet &pset)
4241
: MPFilter(pset),
4342
debug_(pset.getUntrackedParameter<bool>("debug")),
4443
th_option_(pset.getParameter<int>("th_option")),

L1Trigger/DTTriggerPhase2/src/ShowerBuilder.cc

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22
#include "L1Trigger/DTTriggerPhase2/interface/ShowerBuilder.h"
33
#include "FWCore/ParameterSet/interface/ParameterSet.h"
44

5-
using namespace edm;
65
using namespace cmsdt;
76

87
// ============================================================================
98
// Constructors and destructor
109
// ============================================================================
11-
ShowerBuilder::ShowerBuilder(const ParameterSet &pset, edm::ConsumesCollector &iC)
10+
ShowerBuilder::ShowerBuilder(const edm::ParameterSet &pset, edm::ConsumesCollector &iC)
1211
: // Unpack information from pset
1312
showerTaggingAlgo_(pset.getParameter<int>("showerTaggingAlgo")),
1413
threshold_for_shower_(pset.getParameter<int>("threshold_for_shower")),
@@ -19,15 +18,13 @@ ShowerBuilder::ShowerBuilder(const ParameterSet &pset, edm::ConsumesCollector &i
1918
debug_(pset.getUntrackedParameter<bool>("debug")),
2019
scenario_(pset.getParameter<int>("scenario")) {}
2120

22-
ShowerBuilder::~ShowerBuilder() {}
23-
2421
// ============================================================================
2522
// Main methods (initialise, run, finish)
2623
// ============================================================================
2724
void ShowerBuilder::initialise(const edm::EventSetup &iEventSetup) {}
2825

29-
void ShowerBuilder::run(Event &iEvent,
30-
const EventSetup &iEventSetup,
26+
void ShowerBuilder::run(edm::Event &iEvent,
27+
const edm::EventSetup &iEventSetup,
3128
const DTDigiCollection &digis,
3229
ShowerCandidatePtr &showerCandidate_SL1,
3330
ShowerCandidatePtr &showerCandidate_SL3) {

0 commit comments

Comments
 (0)