Skip to content

SiDetectorElement updateCache/updateAllCaches const, make it private

Christos Anastopoulos requested to merge ATLAS-EGamma/athena:SiElement_update_cache_private into master

Mentioning @oda

The idea is :

  • updateCache, updateAllCaches are private and therefore can only be used internally for the lazy initialization
  • For a non-const object one can just use setCache/setAllCaches which are non-const.

If this works....

Then given that updateCache,and updateAllCaches are private one could "relax" a bit the locking as a const client can not call them directly and the setters are not const. i.e one can control better the modification of the cache for const objects.

Edited by Christos Anastopoulos

Merge request reports