From a10abb75a2d494ed3a56c043fe5be3f0f348f2a9 Mon Sep 17 00:00:00 2001 From: Ryuta Kambe Date: Sat, 11 May 2024 19:37:28 +0900 Subject: [PATCH 1/3] perf(behavior_path_dynamic_avoidance_module): use const reference Signed-off-by: Ryuta Kambe --- planning/behavior_path_dynamic_avoidance_module/src/scene.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp index 84871ed756e81..764d198670d7b 100644 --- a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp +++ b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp @@ -1803,7 +1803,7 @@ DynamicAvoidanceModule::EgoPathReservePoly DynamicAvoidanceModule::calcEgoPathRe [&ego_path_lines]( strategy::distance_asymmetric path_expand_strategy, strategy::distance_asymmetric steer_expand_strategy, - std::vector outer_body_path) -> tier4_autoware_utils::Polygon2d { + const std::vector & outer_body_path) -> tier4_autoware_utils::Polygon2d { // reserve area based on the reference path tier4_autoware_utils::MultiPolygon2d path_poly; boost::geometry::buffer( From 85b8fbf4e1f985b0f0a9d20384f38453eb496a03 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Sat, 11 May 2024 10:40:50 +0000 Subject: [PATCH 2/3] style(pre-commit): autofix --- .../src/scene.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp index 764d198670d7b..5b0fd4ca6d0c0 100644 --- a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp +++ b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp @@ -1799,11 +1799,11 @@ DynamicAvoidanceModule::EgoPathReservePoly DynamicAvoidanceModule::calcEgoPathRe ego_path_lines.push_back(tier4_autoware_utils::fromMsg(path_point.point.pose.position).to_2d()); } - auto calcReservePoly = - [&ego_path_lines]( - strategy::distance_asymmetric path_expand_strategy, - strategy::distance_asymmetric steer_expand_strategy, - const std::vector & outer_body_path) -> tier4_autoware_utils::Polygon2d { + auto calcReservePoly = [&ego_path_lines]( + strategy::distance_asymmetric path_expand_strategy, + strategy::distance_asymmetric steer_expand_strategy, + const std::vector & outer_body_path) + -> tier4_autoware_utils::Polygon2d { // reserve area based on the reference path tier4_autoware_utils::MultiPolygon2d path_poly; boost::geometry::buffer( From a0254f3ef00cb1544374d20c9a6dc714bbdb9545 Mon Sep 17 00:00:00 2001 From: Yuki TAKAGI <141538661+yuki-takagi-66@users.noreply.github.com> Date: Mon, 13 May 2024 11:51:34 +0900 Subject: [PATCH 3/3] add const to the other args --- planning/behavior_path_dynamic_avoidance_module/src/scene.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp index 5b0fd4ca6d0c0..a5368732ef3bb 100644 --- a/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp +++ b/planning/behavior_path_dynamic_avoidance_module/src/scene.cpp @@ -1800,8 +1800,8 @@ DynamicAvoidanceModule::EgoPathReservePoly DynamicAvoidanceModule::calcEgoPathRe } auto calcReservePoly = [&ego_path_lines]( - strategy::distance_asymmetric path_expand_strategy, - strategy::distance_asymmetric steer_expand_strategy, + const strategy::distance_asymmetric path_expand_strategy, + const strategy::distance_asymmetric steer_expand_strategy, const std::vector & outer_body_path) -> tier4_autoware_utils::Polygon2d { // reserve area based on the reference path