From 13b51c86d50dcb095061b3ddd4bdc8b16099d845 Mon Sep 17 00:00:00 2001
From: christos <christos@cern.ch>
Date: Wed, 1 Jul 2020 01:52:48 +0100
Subject: [PATCH] Remove default implemenation of EventContext aware method
from IVertexFinder interface
---
.../InDetRecToolInterfaces/IVertexFinder.h | 29 +++++++------------
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/InnerDetector/InDetRecTools/InDetRecToolInterfaces/InDetRecToolInterfaces/IVertexFinder.h b/InnerDetector/InDetRecTools/InDetRecToolInterfaces/InDetRecToolInterfaces/IVertexFinder.h
index 353a12f184d..3dc98dde887 100755
--- a/InnerDetector/InDetRecTools/InDetRecToolInterfaces/InDetRecToolInterfaces/IVertexFinder.h
+++ b/InnerDetector/InDetRecTools/InDetRecToolInterfaces/InDetRecToolInterfaces/IVertexFinder.h
@@ -55,35 +55,28 @@ public:
virtual ~IVertexFinder(){};
static const InterfaceID& interfaceID() { return IID_IVertexFinder; }
- /*
- * For MT we have 2 sets , one with EventContext and one without
- * Implementation really need to overload only one
- * The clients can call either
+ /** Find vertex from Trk::TrackCollection.
+ * @param EventContext
+ * @param input track container
+ * @return a pair of newly created container and auxiliary store
*/
-
virtual std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*>
- findVertex(const EventContext& ctx, const TrackCollection* trackTES) const
- {
-
- (void)(ctx); // We do not use ctx
- return findVertex(trackTES);
- }
+ findVertex(const EventContext& ctx,
+ const TrackCollection* trackTES) const = 0;
/** Find vertex from xAOD::TrackParticleContainer.
* @param EventContext
- * @param trackParticles input track container
+ * @param input track particle container
* @return a pair of newly created container and auxiliary store
*/
virtual std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*>
findVertex(const EventContext& ctx,
- const xAOD::TrackParticleContainer* trackParticles) const
- {
- (void)(ctx); // We do not use ctx
- return findVertex(trackParticles);
- }
+ const xAOD::TrackParticleContainer* trackParticles) const = 0;
/*
- * Non Event context aware methods
+ * Non Event context aware methods.
+ * Can be remove when all clients move to EventContext
+ * aware calls
*/
virtual std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*>
--
GitLab