- The Linux binaries for the 1.1 release were built on the wrong branch, and so include none of the v1.1 changes. This patch fixes some minor compiler errors and includes properly updated binaries.
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
- Removed dependency on C client.
- The plugin now responds to client requests on the same port used to establish the connection.
Removed C client dependency from plugin.
- Replaces the XPCMessage struct.
- Messages are now read from a UDPSocket by the Message class.
- Printing messages to the log is now handled by the Message class.
- In the future, may extend this class with command-specific subclasses that encapsulate additional message parsing functionality.
