From bed8f27f6608deeaa3b4e58e1e453cf6eec81714 Mon Sep 17 00:00:00 2001
From: scott snyder <snyder@bnl.gov>
Date: Tue, 24 Oct 2017 14:59:17 +0200
Subject: [PATCH] SiSpacePointFormation: Fix usage of WriteHandle.
Do not attempt to dereference a WriteHandle that has not been written to.
Fixes ATN test failures in InDetMonitoringATN.
---
.../src/SiTrackerSpacePointFinder.cxx | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/InnerDetector/InDetRecAlgs/SiSpacePointFormation/src/SiTrackerSpacePointFinder.cxx b/InnerDetector/InDetRecAlgs/SiSpacePointFormation/src/SiTrackerSpacePointFinder.cxx
index b06223456b8..7c855cb211c 100755
--- a/InnerDetector/InDetRecAlgs/SiSpacePointFormation/src/SiTrackerSpacePointFinder.cxx
+++ b/InnerDetector/InDetRecAlgs/SiSpacePointFormation/src/SiTrackerSpacePointFinder.cxx
@@ -353,8 +353,12 @@ StatusCode SiTrackerSpacePointFinder::execute_r (const EventContext& ctx) const
{
ATH_MSG_DEBUG( spacepointoverlapCollection->size() <<" overlap space points registered." );
}
- m_numberOfPixel += spacePointContainerPixel->numberOfCollections();
- m_numberOfSCT += spacePointContainer_SCT->numberOfCollections();
+ if (m_selectPixels) {
+ m_numberOfPixel += spacePointContainerPixel->numberOfCollections();
+ }
+ if (m_selectSCTs) {
+ m_numberOfSCT += spacePointContainer_SCT->numberOfCollections();
+ }
if(m_cachemode)//Prevent unnecessary atomic counting
{
m_sctCacheHits += sctCacheCount;
--
GitLab