fix(autoware_behavior_path_start_planner_module): fix duplicateBreak …

…warning (#7583) Signed-off-by: Ryuta Kambe <[email protected]>
autowarefoundation · Jun 19, 2024 · 8dcddae · 8dcddae
1 parent 0eefe2d
commit 8dcddae
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/...ior_path_planner/autoware_behavior_path_start_planner_module/src/start_planner_module.cpp b/...ior_path_planner/autoware_behavior_path_start_planner_module/src/start_planner_module.cpp
@@ -888,20 +888,16 @@ PriorityOrder StartPlannerModule::determinePriorityOrder(
         order_priority.emplace_back(i, planner);
       }
     }
-    return order_priority;
-  }
-
-  if (search_priority == "short_back_distance") {
+  } else if (search_priority == "short_back_distance") {
     for (size_t i = 0; i < start_pose_candidates_num; i++) {
       for (const auto & planner : start_planners_) {
         order_priority.emplace_back(i, planner);
       }
     }
-    return order_priority;
+  } else {
+    RCLCPP_ERROR(getLogger(), "Invalid search_priority: %s", search_priority.c_str());
+    throw std::domain_error("[start_planner] invalid search_priority");
   }
-
-  RCLCPP_ERROR(getLogger(), "Invalid search_priority: %s", search_priority.c_str());
-  throw std::domain_error("[start_planner] invalid search_priority");
   return order_priority;
 }