Use break instead of continue for exiting set cost loops

carlosluis · carlosluis · commit 94aac6d8e873 · 2020-08-05T21:34:49.000+02:00
diff --git a/smac_planner/include/smac_planner/costmap_downsampler.hpp b/smac_planner/include/smac_planner/costmap_downsampler.hpp
@@ -162,12 +162,12 @@ class CostmapDownsampler
     for (int i = 0; i < _downsampling_factor; ++i) {
       mx = x_offset + i;
       if (mx >= _size_x) {
-        continue;
+        break;
       }
       for (int j = 0; j < _downsampling_factor; ++j) {
         my = y_offset + j;
         if (my >= _size_y) {
-          continue;
+          break;
         }
         cost = std::max(cost, _costmap->getCost(mx, my));
       }

Original file line number	Diff line number	Diff line change
`@@ -162,12 +162,12 @@ class CostmapDownsampler`
`162`	`162`	`for (int i = 0; i < _downsampling_factor; ++i) {`
`163`	`163`	`mx = x_offset + i;`
`164`	`164`	`if (mx >= _size_x) {`
`165`		`- continue;`
	`165`	`+ break;`
`166`	`166`	`}`
`167`	`167`	`for (int j = 0; j < _downsampling_factor; ++j) {`
`168`	`168`	`my = y_offset + j;`
`169`	`169`	`if (my >= _size_y) {`
`170`		`- continue;`
	`170`	`+ break;`
`171`	`171`	`}`
`172`	`172`	`cost = std::max(cost, _costmap->getCost(mx, my));`
`173`	`173`	`}`