Calo Cell children override,final un-needed virtual
LarCell/TileCell use virtual (or rm it) , override, final
- LarCell is just a few final (so avoid virtual calls if the compiler knows the concrete type)
- TileCell had quite a few methods that were marked virtual for no clear reason aka they do not
override
neither seem to get everoverriden
. Beyond that we can also mark thigs as final so the compiler can avoid virtual calls if it knows the type. - Leave the
_novirt
(which are there as I guess using the "without reason" virtual ones was slow ...). In principle this we could clean up after this MR. - No clear why the Simul side was inheriting naively it seems to use the default
setGain
fromCaloCell
[https://acode-browser1.usatlas.bnl.gov/lxr/ident?_i=setGain]
Edited by Christos Anastopoulos