From 417d172a721709a1256c6d84704311172241afa1 Mon Sep 17 00:00:00 2001 From: Lorenz Meier Date: Sat, 7 Mar 2020 17:03:51 +0100 Subject: [PATCH] Navigator: Warn only for non-zero missions Previously a zero-length mission that failed checks (e.g. because a mandatory element was not present) would lead to a warning. --- src/modules/navigator/mission.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/modules/navigator/mission.cpp b/src/modules/navigator/mission.cpp index ed0de7aabd..e09d5948c2 100644 --- a/src/modules/navigator/mission.cpp +++ b/src/modules/navigator/mission.cpp @@ -509,11 +509,15 @@ Mission::update_mission() } if (failed) { + // only warn if the check failed on merit + if ((int)_mission.count > 0) { + PX4_WARN("mission check failed"); + } + + // reset the mission _mission.count = 0; _mission.current_seq = 0; _current_mission_index = 0; - - PX4_ERR("mission check failed"); } // find and store landing start marker (if available)