auto_ptr to unique_ptr (7) ATLASRECTS-5296

Jenkins please retry a build

This merge request affects 33 packages. Since this is a long list, I will not print it here.

Adding @goetz ,@harkusha ,@cohm ,@krumnack ,@maklein ,@solodkov ,@szambito ,@ykeisuke ,@jojungge ,@yoyamagu ,@rbielski ,@ggach ,@zmarshal ,@fsforza ,@pavol ,@oducu ,@mnowak ,@ssnyder ,@oabouzei ,@kwhalen ,@mhodgkin ,@nkoehler ,@afaulkne ,@gartoni ,@rbianchi as watchers

added Analysis Core DQ Database EDM Egamma ForwardDetectors Geometry JetEtmiss LAr Reconstruction SUSYTools Simulation TestBeam Tile Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash 93c95e05)

	Athena	AthSimulation	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11753]

Hi @christos

I have one comment related to the commented code. Could you have a look ?

Thanks.

Xiaozhong (L1)

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 119e60c6 - MR comment on comment?

Compare with previous version

This merge request affects 33 packages. Since this is a long list, I will not print it here.

Adding @goetz ,@harkusha ,@cohm ,@krumnack ,@maklein ,@solodkov ,@szambito ,@ykeisuke ,@jojungge ,@yoyamagu ,@rbielski ,@ggach ,@zmarshal ,@fsforza ,@pavol ,@oducu ,@mnowak ,@ssnyder ,@oabouzei ,@kwhalen ,@mhodgkin ,@nkoehler ,@afaulkne ,@gartoni ,@rbianchi as watchers

added review-pending-level-1 label and removed review-user-action-required label

resolved all threads

CI Result SUCCESS (hash 119e60c6)

	Athena	AthSimulation	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11817]

The changes look good, and CI passes. Approving.

Xiaozhong (L1)

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

merged

mentioned in commit cbad5fe4

added sweep:ignore label