Skip to content
Snippets Groups Projects

Adapt changes made in !2920 to current Gaudi & C++ usage

Merged Adapt changes made in !2920 to current Gaudi & C++ usage
All threads resolved!
Merged Gerhard Raven requested to merge follow-up-to-2920 into master
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

Pipeline #4085305 passed

Pipeline passed for 6e8f4103 on follow-up-to-2920

Merged by Sebastien PonceSebastien Ponce 2 years ago (Jun 13, 2022 8:47am UTC)

Merge details

  • Changes merged into master with 78d3a13a.
  • Deleted the source branch.

Pipeline #4089866 passed

Pipeline passed for 78d3a13a on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading