Resolve ros-navigation#3532: reset i (ros-navigation#3597)

SteveMacenski · enricosutera · commit 8d33a1553f30 · 2024-05-19T23:32:08.000+02:00
Signed-off-by: enricosutera &lt;enricosutera@outlook.com&gt;
diff --git a/nav2_theta_star_planner/src/theta_star.cpp b/nav2_theta_star_planner/src/theta_star.cpp
@@ -248,15 +248,13 @@ void ThetaStar::resetContainers()
 
 void ThetaStar::initializePosn(int size_inc)
 {
-  int i = 0;
-
   if (!node_position_.empty()) {
-    for (; i < size_x_ * size_y_; i++) {
+    for (int i = 0; i < size_x_ * size_y_; i++) {
       node_position_[i] = nullptr;
     }
   }
 
-  for (; i < size_inc; i++) {
+  for (int i = 0; i < size_inc; i++) {
     node_position_.push_back(nullptr);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -248,15 +248,13 @@ void ThetaStar::resetContainers()`
`248`	`248`
`249`	`249`	`void ThetaStar::initializePosn(int size_inc)`
`250`	`250`	`{`
`251`		`- int i = 0;`
`252`		`-`
`253`	`251`	`if (!node_position_.empty()) {`
`254`		`- for (; i < size_x_ * size_y_; i++) {`
	`252`	`+ for (int i = 0; i < size_x_ * size_y_; i++) {`
`255`	`253`	`node_position_[i] = nullptr;`
`256`	`254`	`}`
`257`	`255`	`}`
`258`	`256`
`259`		`- for (; i < size_inc; i++) {`
	`257`	`+ for (int i = 0; i < size_inc; i++) {`
`260`	`258`	`node_position_.push_back(nullptr);`
`261`	`259`	`}`
`262`	`260`	`}`