Skip to content
Snippets Groups Projects

GSF . clang-tidy checks. Including possible use after move bugfix

Perhaps the most importand is this

@@ -981,7 +984,8 @@ Trk::GaussianSumFitter::stepForwardFit(
     const Trk::MultiComponentStateOnSurface* multiComponentStateOnSurface =
       new MultiComponentStateOnSurface(
         measurement.release(),
-        Trk::MultiComponentStateHelpers::toPtr(std::move(extrapolatedState)).release(),
+        //used below for the updated state so clone
+        Trk::MultiComponentStateHelpers::clone(extrapolatedState).release(),

as was a possible use after move.

Edited by Christos Anastopoulos

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading