Add ATLAS_CHECK_THREAD_SAFETY to CaloDetDescr

This MR adds ATLAS_CHECK_THREAD_SAFETY to CaloDetDescr

• Main issue was the static s_instance used by CaloDetDercrManager::Intance(). In previous MR have migrated some clients to retrieving it from the store directly and passing it down the call chain. The method still returns an instance but does the full retrieval and does not use a static

• The Align tools had not thread safe callbacks / const_casts. Mark as ATLAS_NOT_THREAD_SAFE

• Furthermore a bit of tidying (from clang), i.e use nullptr add delete to hidden constructors etc.