Modernize CaloFuture
- follow changes in match interface -- no need to add std::optional around a type which has a recognizable invalid state
- put tools/algorithms in namespace Calo::Future
- prefer ToolHandles
- remove use of redundant futuredetails::to_string
- prefer free-standing functions over member functions which do not interact with their hosting class
- remove some unused code
- prefer std::is_base_of_v<...> over std::is_base_of<...>::value
- prefer STL algorithms over raw loops
Must be applied in conjunction with LHCb!1990 (merged)
Edited by Marco Cattaneo
Merge request reports
Activity
Filter activity
Please register or sign in to reply