From bd1b0cab738509d8cc0c9d9fa81e75fa49398af7 Mon Sep 17 00:00:00 2001 From: Silvan Fuhrer Date: Fri, 22 Apr 2022 11:09:39 +0200 Subject: [PATCH] Mission: don't do anything in set_current_mission_index() when index=current already Signed-off-by: Silvan Fuhrer --- src/modules/navigator/mission.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/modules/navigator/mission.cpp b/src/modules/navigator/mission.cpp index 5393dbc56d..3f8951e7c9 100644 --- a/src/modules/navigator/mission.cpp +++ b/src/modules/navigator/mission.cpp @@ -307,8 +307,10 @@ Mission::on_active() bool Mission::set_current_mission_index(uint16_t index) { - if (_navigator->get_mission_result()->valid && - (index != _current_mission_index) && (index < _mission.count)) { + if (index == _current_mission_index) { + return true; // nothing to do, so return true + + } else if (_navigator->get_mission_result()->valid && (index < _mission.count)) { _current_mission_index = index;