Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 192
    • Merge requests 192
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !42008

Closed
Created Mar 25, 2021 by Christos Anastopoulos@christosDeveloper
  • Report abuse
Report abuse

Draft: A first try for adding an EventContext aware shallowCopy. The bit clunky part...

  • Overview 16
  • Commits 3
  • Pipelines 2
  • Changes 3

A first try for adding an EventContext aware shallowCopy.

The bit clunky part is the AnalysisBase cases where we do not really have an EventContext that can be used.

We move the implementation in one common functions Then we have

  • Function accepting cont and EventContext, this is available only for Athena, returning smart ptr.
  • Function accepting cont and ptr to EventContext, this is usable from AnalysisBase , although the ptr is nullptr there. Returns smart ptr.
  • The existing function signature no smart ptr for now (Compatibility).

If at some stage we have a complete EventContext type and we can create one we can make the one with ref also dual-use.

Christos

Edited Mar 26, 2021 by Christos Anastopoulos
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: ShallowCopyEventContext_aware