Modernize UT
- add [[nodiscard]]
- add constexpr
- add helper functions to deal with masked quantities
- add some asserts
- remove unneccessary destructors
- prefer STL algorithms
- have setters return reference to this
- prefer std::nearbyint over std::nearbyintf
- prefer private over protected
- prefer range-based loops
Edited by Gerhard Raven
Merge request reports
Activity
added modernisation label
added 4 commits
-
4d37f6a6...a9d5f64a - 3 commits from branch
master
- 82d2fe96 - Modernize UT
-
4d37f6a6...a9d5f64a - 3 commits from branch
mentioned in issue Moore#273 (closed)
assigned to @ascarabo
mentioned in issue Moore#278 (closed)
assigned to @axu
unassigned @ascarabo
- Resolved by Gerhard Raven
/ci-test --merge
added ci-test-triggered label
- [2021-05-12 09:54] Validation started with lhcb-master-mr#2302
- [2021-05-15 11:44] Validation started with lhcb-master-mr#2313
- [2021-05-15 13:55] Validation started with lhcb-master-mr#2314
Edited by Software for LHCbadded 13 commits
-
9bcb5fed...1c996bbd - 12 commits from branch
master
- 4fbabf51 - Modernize UT
-
9bcb5fed...1c996bbd - 12 commits from branch
Please register or sign in to reply