Merge pull request #43671 from Dr15Jones/cleanupSimTrackerTrackTrigge…

…rAssociation Removed beginRun/endRun methods in TrackTriggerAssociation modules
cms-sw · Jan 12, 2024 · b23651f · b23651f
2 parents 67a371d + 77bd4fb
commit b23651f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 47 deletions.
diff --git a/SimTracker/TrackTriggerAssociation/plugins/TTClusterAssociator.h b/SimTracker/TrackTriggerAssociation/plugins/TTClusterAssociator.h
@@ -47,9 +47,6 @@ class TTClusterAssociator : public edm::stream::EDProducer<> {
   /// Constructors
   explicit TTClusterAssociator(const edm::ParameterSet& iConfig);
 
-  /// Destructor
-  ~TTClusterAssociator() override;
-
 private:
   /// Data members
   edm::Handle<edm::DetSetVector<PixelDigiSimLink> > thePixelDigiSimLinkHandle_;
@@ -64,8 +61,6 @@ class TTClusterAssociator : public edm::stream::EDProducer<> {
   edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> theTrackerGeometryToken_;
 
   /// Mandatory methods
-  void beginRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
-  void endRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
   void produce(edm::Event& iEvent, const edm::EventSetup& iSetup) override;
 
 };  /// Close class
@@ -93,23 +88,11 @@ TTClusterAssociator<T>::TTClusterAssociator(const edm::ParameterSet& iConfig) {
   }
 
   theTrackerGeometryToken_ = esConsumes();
-}
-
-/// Destructor
-template <typename T>
-TTClusterAssociator<T>::~TTClusterAssociator() {}
 
-/// Begin run
-template <typename T>
-void TTClusterAssociator<T>::beginRun(const edm::Run& run, const edm::EventSetup& iSetup) {
   /// Print some information when loaded
   edm::LogInfo("TTClusterAssociator< ") << templateNameFinder<T>() << " > loaded.";
 }
 
-/// End run
-template <typename T>
-void TTClusterAssociator<T>::endRun(const edm::Run& run, const edm::EventSetup& iSetup) {}
-
 /// Implement the producer
 template <>
 void TTClusterAssociator<Ref_Phase2TrackerDigi_>::produce(edm::Event& iEvent, const edm::EventSetup& iSetup);

diff --git a/SimTracker/TrackTriggerAssociation/plugins/TTStubAssociator.h b/SimTracker/TrackTriggerAssociation/plugins/TTStubAssociator.h
@@ -48,9 +48,6 @@ class TTStubAssociator : public edm::stream::EDProducer<> {
   /// Constructors
   explicit TTStubAssociator(const edm::ParameterSet& iConfig);
 
-  /// Destructor
-  ~TTStubAssociator() override;
-
 private:
   /// Data members
   std::vector<edm::InputTag> ttStubsInputTags_;
@@ -63,8 +60,6 @@ class TTStubAssociator : public edm::stream::EDProducer<> {
   edm::ESGetToken<TrackerTopology, TrackerTopologyRcd> theTrackerTopologyToken_;
 
   /// Mandatory methods
-  void beginRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
-  void endRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
   void produce(edm::Event& iEvent, const edm::EventSetup& iSetup) override;
 
 };  /// Close class
@@ -94,23 +89,11 @@ TTStubAssociator<T>::TTStubAssociator(const edm::ParameterSet& iConfig) {
 
   theTrackerGeometryToken_ = esConsumes();
   theTrackerTopologyToken_ = esConsumes();
-}
-
-/// Destructor
-template <typename T>
-TTStubAssociator<T>::~TTStubAssociator() {}
 
-/// Begin run
-template <typename T>
-void TTStubAssociator<T>::beginRun(const edm::Run& run, const edm::EventSetup& iSetup) {
   /// Print some information when loaded
   edm::LogInfo("TTStubAssociator< ") << templateNameFinder<T>() << " > loaded.";
 }
 
-/// End run
-template <typename T>
-void TTStubAssociator<T>::endRun(const edm::Run& run, const edm::EventSetup& iSetup) {}
-
 /// Implement the producer
 template <>
 void TTStubAssociator<Ref_Phase2TrackerDigi_>::produce(edm::Event& iEvent, const edm::EventSetup& iSetup);

diff --git a/SimTracker/TrackTriggerAssociation/plugins/TTTrackAssociator.h b/SimTracker/TrackTriggerAssociation/plugins/TTTrackAssociator.h
@@ -59,8 +59,6 @@ class TTTrackAssociator : public edm::stream::EDProducer<> {
   bool TTTrackAllowOneFalse2SStub;
 
   /// Mandatory methods
-  void beginRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
-  void endRun(const edm::Run& run, const edm::EventSetup& iSetup) override;
   void produce(edm::Event& iEvent, const edm::EventSetup& iSetup) override;
 
 };  /// Close class
@@ -90,22 +88,13 @@ TTTrackAssociator<T>::TTTrackAssociator(const edm::ParameterSet& iConfig) {
 
     produces<TTTrackAssociationMap<T> >(iTag.instance());
   }
-}
-
-/// Destructor
-template <typename T>
-TTTrackAssociator<T>::~TTTrackAssociator() {}
-
-/// Begin run
-template <typename T>
-void TTTrackAssociator<T>::beginRun(const edm::Run& run, const edm::EventSetup& iSetup) {
   /// Print some information when loaded
   edm::LogInfo("TTStubAssociator< ") << "TTTrackAssociator< " << templateNameFinder<T>() << " > loaded.";
 }
 
-/// End run
+/// Destructor
 template <typename T>
-void TTTrackAssociator<T>::endRun(const edm::Run& run, const edm::EventSetup& iSetup) {}
+TTTrackAssociator<T>::~TTTrackAssociator() {}
 
 /// Implement the producer
 template <>