AC_PrecLand: stop libraries including AP_Logger.h in .h files

AP_Logger.h is a nexus of includes; while this is being improved over
time, there's no reason for the library headers to include AP_Logger.h
as the logger itself is access by singleton and the structures are in
LogStructure.h

This necessitated moving The PID_Info structure out of AP_Logger's
namespace.  This cleans up a pretty nasty bit - that structure is
definitely not simply used for logging, but also used to pass pid
information around to controllers!

There are a lot of patches in here because AP_Logger.h, acting as a
nexus, was providing transitive header file inclusion in many (some
unlikely!) places.
This commit is contained in:
Peter Barker 2022-03-04 14:29:50 +11:00 committed by Peter Barker
parent 4db1e6a914
commit c452002ae4
1 changed files with 1 additions and 0 deletions

View File

@ -7,6 +7,7 @@
#include "AC_PrecLand_IRLock.h" #include "AC_PrecLand_IRLock.h"
#include "AC_PrecLand_SITL_Gazebo.h" #include "AC_PrecLand_SITL_Gazebo.h"
#include "AC_PrecLand_SITL.h" #include "AC_PrecLand_SITL.h"
#include <AP_Logger/AP_Logger.h>
#include <GCS_MAVLink/GCS.h> #include <GCS_MAVLink/GCS.h>
extern const AP_HAL::HAL& hal; extern const AP_HAL::HAL& hal;