fix(autoware_path_sampler): fix passedByValue (autowarefoundation#8216)

fix:passedByValue Signed-off-by: kobayu858 <yutaro.kobayashi@tier4.jp>
a-maumau · Jul 26, 2024 · 95aef0d · 95aef0d
1 parent c4aea4b
commit 95aef0d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...ed_planner/autoware_path_sampler/include/autoware_path_sampler/utils/trajectory_utils.hpp b/...ed_planner/autoware_path_sampler/include/autoware_path_sampler/utils/trajectory_utils.hpp
@@ -140,10 +140,10 @@ size_t findEgoSegmentIndex(
 }
 
 std::vector<TrajectoryPoint> resampleTrajectoryPoints(
-  const std::vector<TrajectoryPoint> traj_points, const double interval);
+  const std::vector<TrajectoryPoint> & traj_points, const double interval);
 
 std::vector<TrajectoryPoint> resampleTrajectoryPointsWithoutStopPoint(
-  const std::vector<TrajectoryPoint> traj_points, const double interval);
+  const std::vector<TrajectoryPoint> & traj_points, const double interval);
 
 template <typename T>
 std::optional<size_t> updateFrontPointForFix(

diff --git a/planning/sampling_based_planner/autoware_path_sampler/src/utils/trajectory_utils.cpp b/planning/sampling_based_planner/autoware_path_sampler/src/utils/trajectory_utils.cpp
@@ -59,7 +59,7 @@ void compensateLastPose(
 }
 
 std::vector<TrajectoryPoint> resampleTrajectoryPoints(
-  const std::vector<TrajectoryPoint> traj_points, const double interval)
+  const std::vector<TrajectoryPoint> & traj_points, const double interval)
 {
   constexpr bool enable_resampling_stop_point = true;
 
@@ -71,7 +71,7 @@ std::vector<TrajectoryPoint> resampleTrajectoryPoints(
 
 // NOTE: stop point will not be resampled
 std::vector<TrajectoryPoint> resampleTrajectoryPointsWithoutStopPoint(
-  const std::vector<TrajectoryPoint> traj_points, const double interval)
+  const std::vector<TrajectoryPoint> & traj_points, const double interval)
 {
   constexpr bool enable_resampling_stop_point = false;