Fixed:
- Removed trailing whitespace.
- Removed out of date TODO's.
- Added #include<algorithm> to DataManager.cpp
- Reordered includes to include C system headers before C++ system headers.
- Added username to TODO's
- Renamed include guards to use full project name as prefix.
- Made methods and parameters const where possible.
- Made UDPSocket constructor explicit.
Won't Fix:
- snprintf is introduced in C++11.
- Trailing comments at the end of namespace & header guards
- Did not re-inlude headers already included by the matching XPC header.
- localtime is fine since X-Plane is not multithreaded at all on the plugin side.
- Linter confuses #elif with parens as a function call.
- namespace indentation
- Added global position drefs for multiplayer aircraft.
- Added checks for the "don't change" sentinel value to SetPosition, SetOrientation and SetFlaps.
- Reordered validation checks in SetGear to be consistent with the above methods.
- Removed some now redundant "don't change" value checks in the MessageHandlers class.
- The Get method no longer assumes unknown drefs are floats.
- The Get method now properly accounts for the fact that the dref data type is a bitflag.
- Binary drefs are now properly read and packed into `values`.
- This is a rather inefficient process, as it quadruples the number of bytes we have to send out. Should look into better ways to do this in the future.
- The Set method now handles data types correctly.
- The Set method now supports binary datarefs.
- Improved logging of edge cases.
- Added a check to warn about setting read-only datarefs if verbose logging is enabled.
