Skip to content

Surface.h remove last non-thread safe function "'setOwner() const"

Christos Anastopoulos requested to merge ATLAS-EGamma/athena:Surface_rm_setOwner into master
  • Surface rm the last non-thread safe function "setOwner const" hiding a const_cast keep the thread safe version only.
  • The CaloSurfaceBuilder seems to never really needed the const_cast version. We create new elements and then we want to have them in certain position in a vector. Since we create them we can setOwner at that time
  • Other places that needed the const_cast now have it explicitly

Also update the authors to include recent (last year) changes in MT and interface clean--up

Mentioning @sroe

This is the last non MT safe method under TrkSurfaces so after this should be none remaining.

Merge request reports