in the case it wasn't compiled in the return code would be correct.
in the case that the parameter was invalid we would return UNSUPPORTED , which is wrong, it should be DENIED
we have code which tries to handle commands coming in as command long as command int.
Change to rely on that code working, rather than handling both command-long and command-int variants
clang reports this could be a problem when compiling under some EABIs. Remove it from most places as it is just noise, replace with class where we want to avoid including Location.h
this makes the user experience of a first time install nicer for
copters/helis. At the moment the user finds the GCS is not functional,
for example flight mode drop-downs don't display. This can be
confusing as the user doesn't know about the link between displaying
flight modes and the FRAME_CLASS
We decided to remove this after 4.2 was out in favour of
MAV_CMD_DO_SET_HOME which has been available since 2015.
The gcs-maintainers list was notified in Feburary.
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.
The TODO items to actually implement these are almost 6 years old.
Since then these methods have been burning space.
This doesn't even make sense for several vehicles, so a default
implementation which does nothing seems OK.