From 7da35e9d087ee67c5ab28d15587709b6a5049be0 Mon Sep 17 00:00:00 2001 From: Peter Barker Date: Sat, 25 Aug 2018 09:52:34 +1000 Subject: [PATCH] AP_Mission: change while loop out for for loop --- libraries/AP_Mission/AP_Mission.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libraries/AP_Mission/AP_Mission.cpp b/libraries/AP_Mission/AP_Mission.cpp index a3db04dde2..7a98db39c6 100644 --- a/libraries/AP_Mission/AP_Mission.cpp +++ b/libraries/AP_Mission/AP_Mission.cpp @@ -346,10 +346,8 @@ bool AP_Mission::is_nav_cmd(const Mission_Command& cmd) /// accounts for do_jump commands but never increments the jump's num_times_run (advance_current_nav_cmd is responsible for this) bool AP_Mission::get_next_nav_cmd(uint16_t start_index, Mission_Command& cmd) { - uint16_t cmd_index = start_index; - // search until the end of the mission command list - while(cmd_index < (unsigned)_cmd_total) { + for (uint16_t cmd_index = start_index; cmd_index < (unsigned)_cmd_total; cmd_index++) { // get next command if (!get_next_cmd(cmd_index, cmd, false)) { // no more commands so return failure @@ -359,8 +357,6 @@ bool AP_Mission::get_next_nav_cmd(uint16_t start_index, Mission_Command& cmd) if (is_nav_cmd(cmd)) { return true; } - // move on in list - cmd_index++; } // if we got this far we did not find a navigation command