Remove usage of (old) RichKernel histo base classes and general cleanup of RichReadout

Removes all use of RichKernel histo base classes from Rich components in Boole.

Final remove of RichKernel usage (and thus GaudiAlg) will have to wait until DetDesc support is fully removed from Boole, at which point Rich/RichReadout/src/component/legacy/RichDetailedLegacyFrontEndResponsePMT.cpp can be removed which is the last component still using RichKernel.

In addition, there is some general cleanup of the code across the board.

I also fix a number of issues that prevented a DD4HEP build running on DetDesc SIM MC files (and in general where bad memory access patterns so unstable anyway).

I also resolve a number of (very) bad data access patterns that lead to the poor CPU performance reported in #46 (closed)

Closes #46 (closed)