fix(autoware_freespace_planning_algorithms): fix bugprone-errors (#9670)

kobayu858 · web-flow · commit a88e90eb46a1 · 2024-12-20T15:10:09.000+09:00
* fix: bugprone-error

Signed-off-by: kobayu858 &lt;yutaro.kobayashi@tier4.jp&gt;

* fix: bugprone-error

Signed-off-by: kobayu858 &lt;yutaro.kobayashi@tier4.jp&gt;

---------

Signed-off-by: kobayu858 &lt;yutaro.kobayashi@tier4.jp&gt;
diff --git a/planning/autoware_freespace_planning_algorithms/src/abstract_algorithm.cpp b/planning/autoware_freespace_planning_algorithms/src/abstract_algorithm.cpp
@@ -115,7 +115,7 @@ void AbstractPlanningAlgorithm::setMap(const nav_msgs::msg::OccupancyGrid & cost
   std::vector<bool> is_obstacle_table;
   is_obstacle_table.resize(nb_of_cells);
   for (uint32_t i = 0; i < nb_of_cells; ++i) {
-    const int cost = costmap_.data[i];
+    const int cost = costmap_.data[i];  // NOLINT
     if (cost < 0 || planner_common_param_.obstacle_threshold <= cost) {
       is_obstacle_table[i] = true;
     }
diff --git a/planning/autoware_freespace_planning_algorithms/src/astar_search.cpp b/planning/autoware_freespace_planning_algorithms/src/astar_search.cpp
@@ -388,7 +388,7 @@ double AstarSearch::getExpansionDistance(const AstarNode & current_node) const
 double AstarSearch::getSteeringCost(const int steering_index) const
 {
   return planner_common_param_.curve_weight *
-         (abs(steering_index) / planner_common_param_.turning_steps);
+         (static_cast<double>(abs(steering_index)) / planner_common_param_.turning_steps);
 }
 
 double AstarSearch::getSteeringChangeCost(

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ void AbstractPlanningAlgorithm::setMap(const nav_msgs::msg::OccupancyGrid & cost`
`115`	`115`	`std::vector<bool> is_obstacle_table;`
`116`	`116`	`is_obstacle_table.resize(nb_of_cells);`
`117`	`117`	`for (uint32_t i = 0; i < nb_of_cells; ++i) {`
`118`		`- const int cost = costmap_.data[i];`
	`118`	`+ const int cost = costmap_.data[i]; // NOLINT`
`119`	`119`	`if (cost < 0 \|\| planner_common_param_.obstacle_threshold <= cost) {`
`120`	`120`	`is_obstacle_table[i] = true;`
`121`	`121`	`}`
Original file line number	Diff line number	Diff line change
`@@ -388,7 +388,7 @@ double AstarSearch::getExpansionDistance(const AstarNode & current_node) const`
`388`	`388`	`double AstarSearch::getSteeringCost(const int steering_index) const`
`389`	`389`	`{`
`390`	`390`	`return planner_common_param_.curve_weight *`
`391`		`- (abs(steering_index) / planner_common_param_.turning_steps);`
	`391`	`+ (static_cast<double>(abs(steering_index)) / planner_common_param_.turning_steps);`
`392`	`392`	`}`
`393`	`393`
`394`	`394`	`double AstarSearch::getSteeringChangeCost(`