Skip to content
Snippets Groups Projects

Ensure that derived conditions used in DetDesc comply to DD4hep requirements

Merged Marco Clemencic requested to merge clemenci-master-patch-60188 into master
All threads resolved!

Classes used as derived conditions in DD4hep must be copy/move assignable and default constructible. DetDesc does not have the limitation so the compilation might fail when switching to DD4hep.

This MR adds a static check to ensure that derived conditions have the same constraints in both cases. The static check is also added to DD4hep conditions derivation to make the error more clear (instead of some template error message stating the some methods are not available).

Requires !3543 (merged) Rec!2883 (merged) Boole!397 (merged)

Edited by Marco Clemencic

Merge request reports

Merge request pipeline #4057089 passed

Merge request pipeline passed for 4dec2ebd

Merged by Sebastien PonceSebastien Ponce 2 years ago (Jun 6, 2022 4:03pm UTC)

Loading

Pipeline #4062470 passed

Pipeline passed for ca039fbc 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