Adapt changes made in !2920 to current Gaudi & C++ usage
All threads resolved!
All threads resolved!
- use extend_interfaces and extend, drop explict constructor
- prefer constexpr over preprocessor
- prefer constexpr variables anonymous namespace over class static constexpr
- amalgamate component header into component source file
- proper use of
span
, i.e. pass by value, not reference
Edited by Gerhard Raven
Merge request reports
Activity
added cleanup label
changed title from Adapt changes !2920 (merged) to current Gaudi & C++ usage to Adapt changes made in !2920 (merged) to current Gaudi & C++ usage
added 1 commit
- 848a5fdf - Adapt changes !2920 (merged) to current Gaudi usage
added RTA label
added 1 commit
- dfe20ee7 - Adapt changes !2920 (merged) to current Gaudi usage
added 1 commit
- c4b773b7 - Adapt changes !2920 (merged) to current Gaudi usage
- Resolved by Gerhard Raven
Is it ready for testing ?
added ci-test-triggered label
- [2022-06-10 15:00] Validation started with lhcb-master-mr#4710
added 1 commit
- 6e8f4103 - Adapt changes !2920 (merged) to current Gaudi usage
@sponce this looks ready for the mergehammer
mentioned in commit 78d3a13a
mentioned in issue Moore#444 (closed)
Please register or sign in to reply