Skip to content
Snippets Groups Projects

Modernize CaloFuture

Merged 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading