From 0b16c4063e5d58a356701e0b9852d8920d17a3e0 Mon Sep 17 00:00:00 2001 From: Peter Barker Date: Fri, 4 Mar 2022 17:13:21 +1100 Subject: [PATCH] AP_NavEKF3: 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. --- libraries/AP_NavEKF3/AP_NavEKF3_Measurements.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/libraries/AP_NavEKF3/AP_NavEKF3_Measurements.cpp b/libraries/AP_NavEKF3/AP_NavEKF3_Measurements.cpp index 50bc349744..8cfeadeea6 100644 --- a/libraries/AP_NavEKF3/AP_NavEKF3_Measurements.cpp +++ b/libraries/AP_NavEKF3/AP_NavEKF3_Measurements.cpp @@ -4,6 +4,7 @@ #include #include #include +#include /******************************************************** * OPT FLOW AND RANGE FINDER *