Skip to content
Snippets Groups Projects

Relax the SourceLink req. to copy constructible

Merged Paul Gessinger requested to merge relax-sourcelink-copyable-requirement into master

Previously, the SourceLinkConcept required that the type was is_trivially_copyable (i.e. a type can be copied by memcpy). This makes the frameworks SimIdentifier fail the concept requirements, because it's not. This changes the requirement to simply say is_copy_construcible, a much less stringent criterion.

Merge request reports

Pipeline #982894 passed with warnings

Pipeline passed with warnings for d4aac8d6 on relax-sourcelink-copyable-requirement

Test coverage 70.00% (0.00%) from 1 job
Approved by

Merged by Moritz KiehnMoritz Kiehn 5 years ago (Jul 17, 2019 2:25pm UTC)

Merge details

  • Changes merged into master with 5daa7956.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #983409 passed with warnings

Pipeline passed with warnings for 5daa7956 on master

Test coverage 70.00% (0.00%) from 1 job

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