Tidy

[jpswinski]

Major rework of the code to bring it up to clang-tidy and cppcheck standards. Current code in this branch:

• Runs clang-tidy and cppcheck on every compilation unit when configured for Debug (config-debug), and fails the build on any warning
• Compiles in Address Sanitizer when configured for Debug (config-debug)
• Includes legacy, ccsds, and pistache packages when configured for Debug (config-debug)
• Migrated the use of SafeString to std::string and a new thin const char* wrapper in StringLib called FString. SafeString only exists now in the atlas plugin. All string manipulation should first try to use std::string, then FString, and then StringLib with a const char* if nothing else is a good fit.
• The four base templated data structures in SlideRule: List, Dictionary, Ordering, Table, have been updated so that if they are provided a pointer they automatically manage the memory. The managed versions (e.g. MgList, etc.) have been done away with. If any of these data structures contains a pointer type, the object being pointed to will be automatically deleted by the data structure when necessary. Another way to think of this is that adding a member to one of these containers always transfers ownership.

[elidwa]

I reviewed the code. I also ran my performance tests. I found no issues. I committed a minor change, almost cosmetic. Class member functions implemented in header file are implicitly inlined. The 'inline' keyword is redundant. I removed it from a few header files.