Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L LHCb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 193
    • Issues 193
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 48
    • Merge requests 48
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • LHCb
  • Merge requests
  • !3678

Modernize AddressKillerAlg

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Gerhard Raven requested to merge modernize-addresskiller into master Jul 13, 2022
  • Overview 19
  • Commits 1
  • Pipelines 6
  • Changes 1
  • inherit from Gaudi::Algorithm instead of GaudiAlgorithm to remove (amongst others) warnings that this algorithm is not (recognized as being) re-entrant
  • prefer ServiceHandle over bare pointer, explicit property and call to service(...)
  • use traverseTree instead of objectLeaves to make this work with evtStoreSvc (which does not implement objectLeaves because it cannot...)

required for (and motivated by) !3647

Edited Jul 13, 2022 by Gerhard Raven
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: modernize-addresskiller