From 55818f4f0cf600dc1bc8a3f89d4e751e16e3f170 Mon Sep 17 00:00:00 2001
From: AhmedEbrahim <ahmed.a.d.ebrahim@gmail.com>
Date: Wed, 19 Jul 2023 11:24:00 +0300
Subject: [PATCH] fix(obstacle_avoidance_planner): removing unneeded else
 statement for early return

Signed-off-by: AhmedEbrahim <ahmed.a.d.ebrahim@gmail.com>
---
 planning/obstacle_avoidance_planner/src/node.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/planning/obstacle_avoidance_planner/src/node.cpp b/planning/obstacle_avoidance_planner/src/node.cpp
index 2101be4511524..b37ba62b4294f 100644
--- a/planning/obstacle_avoidance_planner/src/node.cpp
+++ b/planning/obstacle_avoidance_planner/src/node.cpp
@@ -456,9 +456,8 @@ void ObstacleAvoidancePlanner::insertZeroVelocityOutsideDrivableArea(
         motion_utils::insertStopPoint(dist_with_margin, optimized_traj_points);
       if (first_outside_idx_with_margin) {
         return *first_outside_idx_with_margin;
-      } else {
-        return *first_outside_idx;
       }
+      return *first_outside_idx;
     }();
 
     publishVirtualWall(optimized_traj_points.at(stop_idx).pose);