From 73637c240eabb3e2acd2389a1a63b9b343963c67 Mon Sep 17 00:00:00 2001
From: scott snyder <snyder@bnl.gov>
Date: Thu, 1 Apr 2021 16:12:23 +0200
Subject: [PATCH] TrackToCalo: Fix clang warning.

Don't use std::move with a non-lvalue.
---
 .../RecoTools/TrackToCalo/src/PreselCaloExtensionBuilderAlg.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Reconstruction/RecoTools/TrackToCalo/src/PreselCaloExtensionBuilderAlg.cxx b/Reconstruction/RecoTools/TrackToCalo/src/PreselCaloExtensionBuilderAlg.cxx
index 36642822b626..0dd600d6be2c 100644
--- a/Reconstruction/RecoTools/TrackToCalo/src/PreselCaloExtensionBuilderAlg.cxx
+++ b/Reconstruction/RecoTools/TrackToCalo/src/PreselCaloExtensionBuilderAlg.cxx
@@ -41,7 +41,7 @@ namespace Trk {
       if (itrk->index() >= cache->size())
         // Handle the case where the above assumption does not hold
         cache->resize(itrk->index() + 1);
-      cache->at(itrk->index()) = std::move(m_particleCaloExtensionTool->caloExtension(ctx, *itrk));
+      cache->at(itrk->index()) = m_particleCaloExtensionTool->caloExtension(ctx, *itrk);
     }
     auto outputHandle = SG::makeHandle(m_outputCacheKey, ctx);
     ATH_CHECK(outputHandle.record(std::move(cache)));
-- 
GitLab