Skip to content

Modernize ICaloFutureHypoTool interface

Gerhard Raven requested to merge modernize-ICaloFutureHypoTool into master
  • put it in the Calo::Future::Interfaces namespace
  • process now takes a range of CaloHypos instead of a single CaloHypo
  • add a backwards compatible process which takes a single CaloHypo, and forwards it to the range-based process
  • remove the redundant operator() which just calls process

Note that for now, process accepts a range of pointers to CaloHypos -- this is to allow process to be invoked with a KeyedContainer. The aim is to change the storage to a std::vector instead, at which point the interface should be updated to accept a range of CaloHypo instead.

The 2nd point above is necessary, but not sufficient, to address Rec#63.

Must be applied in conjunction with Rec!1591 (merged)

Edited by Marco Cattaneo

Merge request reports