Christos Anastopoulos authored 7 months ago and Tadej Novak committed 7 months ago

AtlasGeometryCommon : clang-tidy related fixes

Scott Snyder authored 2 years ago and Tadej Novak committed 2 years ago

BeamPipeGeoModel: Fix memory leaks.

Fix a few (one-time) memory leaks.

DataQualityInterfaces, LArGeoEndcap, DumpEventDataToJSON, BeamPipeGeoModel cppcheck / clang-tidy fixes

Dropped the redundant AbsMaterialManager interface, and removed all references to this
interface from client packages.

Dropped redundant methods from RDBMaterialManager, and in this way got rid of lots of
duplicated code.

Given the standalone library is based on Eigen instead of CLHEP, all GeoModel clients
in Athena code were also migrated to the Eigen-based interface of GeoModel. This affected
a large number of packages. Most significant changes have been introduced to subsystem
GeoModel packages.

The in-house GeoModelKernel code has been removed from the repository.

Almost all header files in PixelGeoModel have been moved to the src directory.
The next step should be turning this package into a component-only library.
Similar code reorganization can be applied to SCT_GeoModel too.

Subsystem readout geometry interfaces remain mostly unchanged, except for a minor
change in the InDet readout geometry interface (which was unavoidable).

Several non-GeoModel packages affected by this migration mostly required addition
of missing CLHEP includes.


Former-commit-id: 9cffebb1

Fix coverity warning: potential memory leak on error path.


Former-commit-id: d56d09047b577f0cb167b4bd7c085c8db375212b

coverity: possible null pointer dereference.

Former-commit-id: e9ee2aef

Coverity warning: possible null pointer dereference.
Just remove the null test here; we're not anyway doing anything useful
if it fails.


Former-commit-id: 030f1040

Vakhtang Tsulaia authored 7 years ago and Adam Edward Barton committed 7 years ago

Former-commit-id: 237a8a9f578e5f8c15889235a5f56275ad8eb16d