Fix use after move described in ATLASRECTS-5738

Closes ATLASRECTS-5738

See merge request atlas/athena!38275

AthContainers: Fix variable selection.

See merge request atlas/athena!38242

L1CaloFEXSim: Fix compilation with clang.

See merge request atlas/athena!38245

TrigServices+L1Decoder+CaloRec: Fix clang warnings.

See merge request atlas/athena!38246

Cleanup memory handling of sTGC_Digitization

See merge request atlas/athena!38234

Update on Run 2 MC TriggerDB oracle->sqlite

See merge request atlas/athena!38264

Cleanup memory handling of MM_DigitizationTool

See merge request atlas/athena!38257

LArByteStream+TrigT2CaloCommon: Const fixes

See merge request atlas/athena!38161

LArRecUtils+LArElecCalib: Enable thread-safety checking.

See merge request atlas/athena!38260

In my previous work to migrate the GlobalChi2Fitter to smart pointers, I
introduced a minor bug-prone bit of code where a smart pointer may be
used after being moved. While this has not been disasterous, it is still
erroneous behaviour that clang-tidy picks up on, as described in
ATLASRECTS-5738. This commit resolves the issue by keeping a non-unique
observer pointer to the object, and using that instead of the unique
handle which may be moved. This is mostly a quick fix, the method
containing this code is extremely complex and needs some refactoring.

Add OS X metadata files starting with ._ to .gitignore

See merge request atlas/athena!38211

Quick L1Calo bitwise simulation software bug fixes

See merge request atlas/athena!38232

Fix uninitialized variable warnings for TRT raw monitoring (also fixes FPEs)

See merge request atlas/athena!38249

TauID+TileTripReader+TrackingID: cmake cleanup

See merge request atlas/athena!38214

GSF: cleanup unused MulticomponentState functions. Also use "move" to avoid  allocations.

See merge request atlas/athena!38251

LwtnnUtils, fix clang llvm 9 on inconsistent use of struct vs class

See merge request atlas/athena!38254

Flow Element validation plots

See merge request atlas/athena!37788

Matthew Thomas Anthony authored 4 years ago and Adam Edward Barton committed 4 years ago

Added relevant class components for FE histogram filling. Compiles, but is not necessarily correct runtime at present

ElectronPhotonID: cmake cleanup

See merge request atlas/athena!38212

TrigConfData: Fix clang warning.

See merge request atlas/athena!38247

IOVSvc: Fix bug in incident listener

See merge request atlas/athena!38230

[ATR-22366] Python bug fix to xAOD trigger menu check

See merge request atlas/athena!38226

TrigEgammaAnalysisTools: Don't print messages at the DEBUG level for TrigEgammaMatchingTool

See merge request atlas/athena!38240

TGCTriggerCondSvc: Fix clang warnings.

See merge request atlas/athena!38263

TrigMuonRoITools+MooSegmentCombinationFinder+TrigEgammaMonitoring: Fix clang warnings.

See merge request atlas/athena!38261

MuonTrackSteeringTools+LArCalibTools: Fix clang warnings.

See merge request atlas/athena!38262

TileMonitoring: Add python script to run online Tile monintoring in Run3

See merge request atlas/athena!38176

Remove some defaulted default ctors that can't actually be defaulted
due to the presence of a const member.

Missing override keywords.

Unused variable.

Unused private data member.

Missing override keywords.

Missing override keywords.

Missing override keyword.

Mark more obsolete / unsafe tool interfaces as not thread-safe:
  ILArADC2MeVTool
  ILArAutoCorrNoiseTool
  ILArAutoCorrTotalTool
  ILArOFCTool

Enable thread-safety checking.
Fix resulting warnings.
Some obsolete tools just marked as not thread-safe.