Merge branch 'somefixes' into 'master'

Fix unnecessary copies

See merge request atlas/athena!27260

MuonSpectrometer/MuonReconstruction/MuonSegmentCombiners/MuonSegmentCombinerTools/MooSegmentCombinationFinder/src/MooSegmentFinderAlg.cxx

@@ -110,12 +110,12 @@ StatusCode MooSegmentFinderAlg::execute()
   m_segmentFinder->findSegments( mdtCols, cscCols, tgcCols, rpcCols, output );
 
   if(output.patternCombinations){
-    if( patHandle.record(std::make_unique<MuonPatternCombinationCollection>(std::move(*(output.patternCombinations)))).isSuccess() ){
+    if( patHandle.record(std::unique_ptr<MuonPatternCombinationCollection>(output.patternCombinations)).isSuccess() ){
       ATH_MSG_VERBOSE("stored MuonPatternCombinationCollection at " << m_patternCombiLocation.key());
     }else{
       ATH_MSG_ERROR("Failed to store MuonPatternCombinationCollection at " << m_patternCombiLocation.key());
     }
-    delete output.patternCombinations;
+    output.patternCombinations = nullptr;
   }
   else{
     if( patHandle.record(std::make_unique<MuonPatternCombinationCollection>()).isSuccess() ){

Trigger/TrigAlgorithms/TrigT2BeamSpot/src/T2VertexBeamSpotTool.cxx

@@ -358,11 +358,11 @@ void T2VertexBeamSpotTool::reconstructVertices( ConstDataVector<TrackCollection>
       // Extract beam spot parameters
       SG::ReadCondHandle<InDet::BeamSpotData> beamSpotHandle { m_beamSpotKey };
 
-      auto beamSpot = std::make_unique<T2BeamSpot>(*beamSpotHandle);
+      T2BeamSpot beamSpot(*beamSpotHandle);
 
-      ATH_MSG_DEBUG( "Beamspot from BeamCondSvc: " << *beamSpot); 
+      ATH_MSG_DEBUG( "Beamspot from BeamCondSvc: " << beamSpot);
 
-      const T2Vertex myVertex( *primaryVertex, vertexTracks.get(), *beamSpot, m_trackClusterer->seedZ0() );
+      const T2Vertex myVertex( *primaryVertex, vertexTracks.get(), beamSpot, m_trackClusterer->seedZ0() );
 
       // Monitor all vertices parameters
       monitor_vertex( "Vertex", "", myVertex );