Mostly these are inconsistent usage of int vs. unsigned int vs. long vs.
size_t, plus one abstract base class that needed a virtual destructor
and a few logical expressions for which clang considers parentheses to
be necessary for clarity.
Also fix a couple of issues found by clang-tidy, mostly related to
narrowing integer conversions. Avoided touching things that might
affect the API or floating point results.
Mostly these are inconsistent usage of int vs. unsigned int vs. long vs. size_t, plus one abstract base class that needed a virtual destructor and a few logical expressions for which clang considers parentheses to be necessary for clarity.
Also fix a couple of issues found by clang-tidy, mostly related to narrowing integer conversions. Avoided touching things that might affect the API or floating point results.