Closes ATEAM-677

 - Pass class instances by const reference, not by value.
 - Prefer using an initializer list to assigning members
   in a ctor body.
 - Prefer preincrement (or range for) to postincrement for iterators.
 - Forbid assignment where copy is already forbidden.
 

Enable thread-safety checker.
Clean up / suppress warnings as appropriate.

Remove atlas_depends_on_subdirs and update link dependencies.

Avoid spurious object copy in range for.

Remove use of undefined ROOT macros.

Geneate dictionaries for types in this package.
Avoids errors with pyroot.
See ATEAM-616.
(Will also need a change in PyDumper to completely fix
the MuonEventAthenaPool issues.)

Adam Edward Barton authored 4 years ago and Frank Winklmeier committed 4 years ago

Virtualize the internal code of the IdentifiableContainer and use standard vector iteration for faster access

Library dependency fixes.

Fix header guard mismatch.