Skip to content

Modernize CaloFuture

Gerhard Raven requested to merge modernize-match into master
  • 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