From 12642b57938aa83e4078394645e1f7b4bba05959 Mon Sep 17 00:00:00 2001 From: Nick Exton Date: Thu, 28 Sep 2023 11:02:31 +1000 Subject: [PATCH] AP_Mission: Remove unused rewind param from set_current_cmd() (NFC) --- libraries/AP_Mission/AP_Mission.cpp | 8 +++----- libraries/AP_Mission/AP_Mission.h | 2 +- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/libraries/AP_Mission/AP_Mission.cpp b/libraries/AP_Mission/AP_Mission.cpp index a000e7ba91..3487d3a974 100644 --- a/libraries/AP_Mission/AP_Mission.cpp +++ b/libraries/AP_Mission/AP_Mission.cpp @@ -538,7 +538,7 @@ int32_t AP_Mission::get_next_ground_course_cd(int32_t default_angle) } // set_current_cmd - jumps to command specified by index -bool AP_Mission::set_current_cmd(uint16_t index, bool rewind) +bool AP_Mission::set_current_cmd(uint16_t index) { // read command to check for DO_LAND_START Mission_Command cmd; @@ -546,10 +546,8 @@ bool AP_Mission::set_current_cmd(uint16_t index, bool rewind) _flags.in_landing_sequence = false; } - // mission command has been set and not as rewind command, don't track history. - if (!rewind) { - reset_wp_history(); - } + // mission command has been set, don't track history. + reset_wp_history(); // sanity check index and that we have a mission if (index >= (unsigned)_cmd_total || _cmd_total == 1) { diff --git a/libraries/AP_Mission/AP_Mission.h b/libraries/AP_Mission/AP_Mission.h index e01c372ffa..587b690e9b 100644 --- a/libraries/AP_Mission/AP_Mission.h +++ b/libraries/AP_Mission/AP_Mission.h @@ -601,7 +601,7 @@ public: } // set_current_cmd - jumps to command specified by index - bool set_current_cmd(uint16_t index, bool rewind = false); + bool set_current_cmd(uint16_t index); // restart current navigation command. Used to handle external changes to mission // returns true on success, false if current nav command has been deleted