Skip to content
GitLab

More const-correct Linked{From,To}

Merged Gerhard Raven requested to merge more-const-correct-linkedtofrom into master
  • remove Linked{From,To} constructor which requires an IDataProviderSvc and which fetches (implicitly!) an LHCb::LinksByKey from the TES. Instead, require the caller to provide the LinksByKey explicitly. As a result, Linked{To,From} can now be seen solely as templated extensions of LinksByKey which provide appropriately typed ranges of linked objects. This is a necessary (but not sufficient) step in order to get code which uses MC association to explicitly state their data flow dependencies (which will enable future simplifications to the read-back configuration). In addition, remove the need to specify the 2nd (and 3rd) template arguments to Linked{From,To} and instead make sure the compiler can and will deduce them implicitly.
  • add const where appropriate
  • change return type of Linked{From,To}::range from a vector to a (lazy) range
  • adapt code using Linked{From,To}, including several interfaces which are now more const-correct.

requires Lbcom!647 (merged), Boole!441 (merged), Rec!3254 (merged), Analysis!941 (merged), MooreAnalysis!101 (merged) and Alignment!339 (merged)

Edited by Gerhard Raven