refactor(mission_planner): remove redundant is_reroute check (#6980)

remove redundant is_reroute check Signed-off-by: Ryuta Kambe <[email protected]>
autowarefoundation · May 11, 2024 · 8176c63 · 8176c63
1 parent c24ad46
commit 8176c63
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/planning/mission_planner/src/mission_planner/mission_planner.cpp b/planning/mission_planner/src/mission_planner/mission_planner.cpp
@@ -250,7 +250,7 @@ void MissionPlanner::on_set_lanelet_route(
 
   if (is_reroute && !check_reroute_safety(*current_route_, route)) {
     cancel_route();
-    change_state(is_reroute ? RouteState::SET : RouteState::UNSET);
+    change_state(RouteState::SET);
     throw service_utils::ServiceException(
       ResponseCode::ERROR_REROUTE_FAILED, "New route is not safe. Reroute failed.");
   }
@@ -298,7 +298,7 @@ void MissionPlanner::on_set_waypoint_route(
 
   if (is_reroute && !check_reroute_safety(*current_route_, route)) {
     cancel_route();
-    change_state(is_reroute ? RouteState::SET : RouteState::UNSET);
+    change_state(RouteState::SET);
     throw service_utils::ServiceException(
       ResponseCode::ERROR_REROUTE_FAILED, "New route is not safe. Reroute failed.");
   }