better memory management in Geo2G4Svc

Manual sweep of !22123 (merged) because of a conflict in ExtParameterisedVolumeBuilder.cxx due to the new header: #include "GeoPrimitives/CLHEPtoEigenConverter.h"

mentioned in merge request !22123 (merged)

unmarked as a Work In Progress

This merge request affects 1 package:

• Simulation/G4Utilities/Geo2G4

Adding @jchapman ,@vpascuzz as watchers

added Simulation master review-pending-level-1 labels

CI Result FAILURE

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-22149-2019-03-27-13-48
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 36034]

Jenkins please retry a build

This merge request affects 1 package:

• Simulation/G4Utilities/Geo2G4

Adding @jchapman ,@vpascuzz as watchers

CI Result FAILURE

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-22149-2019-03-28-21-01
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 36157]

Jenkins please retry a build

This merge request affects 1 package:

• Simulation/G4Utilities/Geo2G4

Adding @jchapman ,@vpascuzz as watchers

INFO: precompile check did NOT find banned words in the changeset

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-22149-2019-04-01-12-52
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 36310]

The changes look fine. Approving -Ruchi (L1)

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 3c02fd7b

added sweep:ignore label