Skip to content

Share common code between MuonStationBuilder and MuonStationBuilderCond

This follows MR!59277

Same idea

  • We create an Impl clas.
  • We move all the common things to it .
  • Note that is NOT new code but copy paste of the "common" code
  • So it has the same "issues", but now in one place not two (so potentially easer to solve in the future?)
  • The 2 derived ones differ on how one "gets" the EventContext
  • So we end up with a single 1500 lines impl, rather than 2 ...

This follows some discussion with @jchapman.

PS: I will need to migrate in a similar mannes another piece of code before...

But prb we will need to change the interface of Trk::IDetachedTrackingVolumeBuilder to Trk::IDetachedTrackingVolumeBuilderCond i.e to return unique_ptr. As here it leads to having to "release" things so as to conform to the existing interface.

This is prb 2 MRs down the road but will be needed before "the final step".

Edited by Christos Anastopoulos

Merge request reports