* fix unsigned int to int comparison
* revert previous mistake
* declaration & init together, sinthetaSq in [0, 1]
* range based for-loops
* switch-statements
* declaration position changed
* declaration position changed
* declaration position changed
* declaration position changed, switch statements
* declaration position changed
* declaration position changed
* change in if statement
* now with const iterators
* declaration at initialization
* convert to static casts
* use switch when possible, no breaks
* Update Code/Geometry/point.h
Co-authored-by: Greg Landrum <greg.landrum@gmail.com>
* change from review
remove redundant PRECONDITION
---------
Co-authored-by: Greg Landrum <greg.landrum@gmail.com>
* run clang-tidy with readability-braces-around-statements
clang-format the results
clean up all the parts that clang-tidy-8 broke
* fix problem on windows
* a round of cleanups courtesy of PVS studio
* add a test to make sure that a warning is a false alarm
* bug fix
* Fix a UFF bug
* more PVS studio cleanups
* next round of PVS studio cleanups
* completely remove the chances for that bug
* changes in response to review
* add an additional test
+ a bit of reformatting that snuck in
* first round of cleanups based on PVS-studio suggestions
* a couple more
* a few more cleanups
* another round of cleanups
* undo one of those cleanups
we want the integer rounding behavior here
* add a comment to make that clear
* Fix for filter catalog PRECONDITION redundancy