Skip to content
Snippets Groups Projects

Make ParameterSet store covariance as optional instead of pointer

Merged Paul Gessinger-Befurt requested to merge parameter-cov-optional into master

This should have the advantage of it being local to the class, rather than a heap allocation that the client has to perform and pass to ParameterSet.

Edited by Paul Gessinger-Befurt

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robert Johannes Langenberg resolved all discussions

    resolved all discussions

  • i like it. since this is a somewhat performance critical piece of code, I think it would be interesting to know any effects on performance of this change.

  • Robert Johannes Langenberg approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading