Most of the current .clang-tidy contained checks:
google-explicit-constructor
google-readability-braces-around-statements
modernize-loop-convert
modernize-use-default-member-init
modernize-use-emplace
modernize-use-using
readability-braces-around-statements
readability-else-after-return
readability-named-parameter
*readability-namespace-comments
*readability-inconsistent-*
readability-redundant-member-init
readability-simplify-boolean-expr
readability-string-compare
*explicit-conversions*

cppcheck message:
Prefer prefix ++/-- operators for non-primitive types.. Prefix ++/--
operators should be preferred for non-primitive types.
Pre-increment/decrement can be more efficient than
post-increment/decrement. Post-increment/decrement usually involves
keeping a copy of the previous value around and adds a little extra
code.

cppcheck message:
class 'Raster' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.

Clang format run on near-the-header lines.

Happy New Year

Karsten Rink authored 7 years ago and Dmitri Naumov committed 7 years ago

* reimplemented raster to mesh conversion

* fixed index issues and added 3d index calculation

* added necessary changes to gui dialogue

* added tests

* clang format

* added ifdef for data explorer/vtk includes

* modified dereferencing of unique pointers, formatting

* changed pointer to vector in vtk prep

* using const& for nodes vec

* fixed remaining occurences of dereferencing raster object

This potentially saves a second copy compared to using const
reference. See Effective Modern C++, Item 41.

changed signature of constructor, Raster::refineRaster(), added documentation
compress vector of source properties

Revert "moved vtk-independent functionality from VtkRaster to Raster and modified existing functionality in Raster so that everything fits."

This reverts commit eede3632.