Merge pull request #4281 from RosettaCommons/roccomoretti/clang_tidy_test_fixes
Fixes for clang-tidy issues
This merge both adds a new test server test which runs clang-tidy on the Rosetta code base, as well as fixing a number of issues which the default clang-tidy settings identify. (Mostly using virtual functions in constructors and destructors, but a smattering of other issues.)
There are a couple of places where the current (C++ standard-mandated) behavior probably isn't what is intended, but I've added comments to those, so hopefully people looking at it in the future can figure out if they want/need to change things.