Skip to content

Commit 20f914c

Browse files
veqccsimon-eisenmann-driveblocks
authored andcommitted
fix(autoware_behavior_path_start_planner_module): fix duplicateBreak warning (autowarefoundation#7583)
Signed-off-by: Ryuta Kambe <ryuta.kambe@tier4.jp> Signed-off-by: Simon Eisenmann <simon.eisenmann@driveblocks.ai>
1 parent 380ed71 commit 20f914c

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

planning/behavior_path_planner/autoware_behavior_path_start_planner_module/src/start_planner_module.cpp

+4-8
Original file line numberDiff line numberDiff line change
@@ -888,20 +888,16 @@ PriorityOrder StartPlannerModule::determinePriorityOrder(
888888
order_priority.emplace_back(i, planner);
889889
}
890890
}
891-
return order_priority;
892-
}
893-
894-
if (search_priority == "short_back_distance") {
891+
} else if (search_priority == "short_back_distance") {
895892
for (size_t i = 0; i < start_pose_candidates_num; i++) {
896893
for (const auto & planner : start_planners_) {
897894
order_priority.emplace_back(i, planner);
898895
}
899896
}
900-
return order_priority;
897+
} else {
898+
RCLCPP_ERROR(getLogger(), "Invalid search_priority: %s", search_priority.c_str());
899+
throw std::domain_error("[start_planner] invalid search_priority");
901900
}
902-
903-
RCLCPP_ERROR(getLogger(), "Invalid search_priority: %s", search_priority.c_str());
904-
throw std::domain_error("[start_planner] invalid search_priority");
905901
return order_priority;
906902
}
907903

0 commit comments

Comments
 (0)