Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • R Rec
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 280
    • Issues 280
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 51
    • Merge requests 51
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • Rec
  • Issues
  • #223
Closed
Open
Issue created Oct 18, 2021 by Gerhard Raven@gravenDeveloper0 of 1 checklist item completed0/1 checklist item

Prefer ADL customization over functions in Sel::Utils namespace

The following discussion from !2568 (merged) should be addressed:

  • @graven started a discussion:

    reminder to self (i.e. turn into an issue): we should change this so that this line can be:

          auto const decay_vertex_position = endVertexPos( composite );

    i.e. any 'composite' should define, in its own namespace, a endVertexPos function -- so that it can be found using ADL. That way we can avoid introducing a 'central' place (in Sel::Utils) which must know about every possible supported type -- and it allows to keep endVertexPos next to the actual code it interacts with instead, i.e. it is a more scalable solution (and one of the reasons ADL and namespaces were introducing into the language in the first place, so why not make use of them...).

Assignee
Assign to
Time tracking