diff options
author | Steven Blass <steven.r.blass@gmail.com> | 2014-11-18 08:26:29 -0500 |
---|---|---|
committer | Steven Blass <steven.r.blass@gmail.com> | 2014-11-18 08:26:29 -0500 |
commit | 4eb930f704018253a4cff3604cb9faee8468283b (patch) | |
tree | 8b4c9dda65e4febb0ebdd4e06f93d7e8e9d8b804 /src/modules/navigator | |
parent | 5f6d03099e07e20abe9cec685fee1244d7c9a0d4 (diff) | |
download | px4-firmware-4eb930f704018253a4cff3604cb9faee8468283b.tar.gz px4-firmware-4eb930f704018253a4cff3604cb9faee8468283b.tar.bz2 px4-firmware-4eb930f704018253a4cff3604cb9faee8468283b.zip |
improved readability of offboard fixes
Diffstat (limited to 'src/modules/navigator')
-rw-r--r-- | src/modules/navigator/navigator.h | 1 | ||||
-rw-r--r-- | src/modules/navigator/navigator_main.cpp | 6 |
2 files changed, 4 insertions, 3 deletions
diff --git a/src/modules/navigator/navigator.h b/src/modules/navigator/navigator.h index d550dcc4c..9cd609955 100644 --- a/src/modules/navigator/navigator.h +++ b/src/modules/navigator/navigator.h @@ -205,6 +205,7 @@ private: bool _can_loiter_at_sp; /**< flags if current position SP can be used to loiter */ bool _pos_sp_triplet_updated; /**< flags if position SP triplet needs to be published */ + bool _pos_sp_triplet_published_invalid_once; /**< flags if position SP triplet has been published once to UORB */ control::BlockParamFloat _param_loiter_radius; /**< loiter radius for fixedwing */ control::BlockParamFloat _param_acceptance_radius; /**< acceptance for takeoff */ diff --git a/src/modules/navigator/navigator_main.cpp b/src/modules/navigator/navigator_main.cpp index 5e9806499..035b0d18c 100644 --- a/src/modules/navigator/navigator_main.cpp +++ b/src/modules/navigator/navigator_main.cpp @@ -137,6 +137,7 @@ Navigator::Navigator() : _gpsFailure(this, "GPSF"), _can_loiter_at_sp(false), _pos_sp_triplet_updated(false), + _pos_sp_triplet_published_invalid_once(false), _param_loiter_radius(this, "LOITER_RAD"), _param_acceptance_radius(this, "ACC_RAD"), _param_datalinkloss_obc(this, "DLL_OBC"), @@ -469,9 +470,8 @@ Navigator::task_main() } /* if nothing is running, set position setpoint triplet invalid once */ - static bool published_once = false; - if (_navigation_mode == nullptr && !published_once) { - published_once = true; + if (_navigation_mode == nullptr && !_pos_sp_triplet_published_invalid_once) { + _pos_sp_triplet_published_invalid_once = true; _pos_sp_triplet.previous.valid = false; _pos_sp_triplet.current.valid = false; _pos_sp_triplet.next.valid = false; |