Skip to content
Snippets Groups Projects

follow LHCb!2099

Merged Gerhard Raven requested to merge modernize-calo into master
  • rename Calo::Future into LHCb::Calo
  • follow changes in CaloFutureUtils
  • prefer string_view where appropriate
  • avoid declaring uninitialized variables
  • prefer std::optional over 'special values'
  • avoid functions with lots of arguments of the same type -- instead define a dedicated struct so the variables can be explicitly named
  • prefer std::copy, std::transform over explicit loop

to be applied in conjunction with LHCb!2099 (merged)

Edited by Marco Cattaneo

Merge request reports

Pipeline #1094178 passed

Pipeline passed for a21fb80a on modernize-calo

Merged by Marco CattaneoMarco Cattaneo 5 years ago (Sep 16, 2019 6:25am UTC)

Loading

Pipeline #1095799 passed

Pipeline passed for 1aef678e 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