The number of times x->y->z.t() is called just hurts my eyes.
considering that allBC is a handle to a View and that access to items goes through making a Ptr just to immediately dereference it (not sure the compiler takes this all out)
- deref the handle at the start
- use const CaloCluster& cc = allBCs[i] or similar