New filter clean up
There were some oversights in previous MRs related to migration of filters to the new interface (including those originally exists in the legacy code). They don't affect the correctness of filters but should be addressed. For example:
-
auto field = ...
=>const auto& field = ...
- move around or inline previous item to prevent accidental lambda capture.
-
[&, this]
=> [&] - Use the
Field
overload ofCastAndCall*Field
. - restructure
contour/worklet
anddensity_estimate/worklet
. -
explicit
keyword for constructors taking a single parameter. - remove default constructor where it is necessary/correct/safe to prevent un-inited data members.
- remove unnecessary #include.
- other issues reported by clang-tidy.
There are many other issues reported by clang-tidy at the Worklet level. Unless they are very obviously true positive, they are left untouched.