Internal cleanup: hit and truth outputs
This cleans up a few things following !182 (merged) and !225 (merged):
- Split the
SingleBTagTools
intoconst SingleBTagTools
andSingleBTagOutputs
. I hope this distinction makes it clear that none of the tools should be mutable in the event loop. At the very least it should and enforceconst
-ness in the future1. - Absorb the
Hit
class into the only file that uses it,HitDecorator.cxx
. The work it was doing was so minimal that making its methods free functions seemed simpler. - Remove the
TruthToolkit
struct
inSingleBTagTools
, replace it withTruthWriter
, which was one of the three original data members ofTruthToolkit
. The other two members were copied out of a configuration object, and were only used to initialize other tools, in a scope where the the original configuration object was still available.
-
For now
SingleBTagTools
is still notconst
, because some methods from Athena were non-const
. This should be reverted once we bump to a release containing atlas/athena!49253 (merged)↩
Edited by Dan Guest