See discussion here:
https://github.com/ArduPilot/ardupilot/issues/7331
we were getting some uninitialised variables. While it only showed up in
AP_SbusOut, it means we can't be sure it won't happen on other objects,
so safest to remove the approach
Thanks to assistance from Lucas, Peter and Francisco
This commit changes the way libraries headers are included in source files:
- If the header is in the same directory the source belongs to, so the
notation '#include ""' is used with the path relative to the directory
containing the source.
- If the header is outside the directory containing the source, then we use
the notation '#include <>' with the path relative to libraries folder.
Some of the advantages of such approach:
- Only one search path for libraries headers.
- OSs like Windows may have a better lookup time.
- If a Rally point is being used, always respect the altitude set by the
user (don't take the max of that and the home point altitude).
- No need for constructor to pass in size of RallyLocation struct
Peter Barker
Siddharth Purohit
Michael du Breuil
Andrew Tridgell
Lucas De Marchi
Mathieu OTHACEHE
Francisco Ferreira
Gustavo Jose de Sousa
Randy Mackay
KiwiHC16
Andrew Tridgell
Michael Day
Andrew Chapman