Modernize UTTools

• prefer inheriting from extends
• prefer Gaudi::Property
• prefer DataObject{Read,Write,}Handle
• prefer ToolHandle
• prefer inherited constructors
• remove redundant destructor declarations
• limit scope of variables as much as possible
• prefer STL algorithms
• prefer range-based for loops
• absorb component header files into their corresponding source file
• prefer monadic StatusCode functions

added backport run2 cleanup modernisation labels

added 1 commit

• 705bc412 - Modernize UTTools

Compare with previous version

/ci-test --merge

added 1 commit

• e387ba94 - Modernize UTTools

Compare with previous version

added 1 commit

• 5d9e88d7 - Modernize UTTools

Compare with previous version

added 1 commit

• db4c5a15 - Modernize UTTools

Compare with previous version

added 1 commit

• 7bede7a0 - Modernize UTTools

Compare with previous version

removed backport run2 label

assigned to @mstahl

approved this merge request

assigned to @rmatev and unassigned @mstahl

resolved all threads

mentioned in issue Moore#128 (closed)

@rmatev this looks good. Shall we just test it again since it has been awhile?

added lhcb-gaudi-head label

mentioned in merge request LHCb!2298 (merged)

Nightlies happy so I think this is ready to merge @rmatev.

removed lhcb-gaudi-head label

@rmatev: what is the plan for this MR? @mstahl wrote "I would say it's ready to be merged" a month ago, and @rjhunter wrote a week ago "I think this is ready to merge"....

mentioned in issue Moore#136 (closed)