Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • 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 124
    • Merge requests 124
  • 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
  • !29791

add ATLAS_CHECK_THREAD_SAFETY to CaloClusterCorrection

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Christos Anastopoulos requested to merge ATLAS-EGamma/athena:CaloClusterCorrection_Check_thread into master Jan 25, 2020
  • Overview 2
  • Commits 1
  • Pipelines 1
  • Changes 10

Add ATLAS_CHECK_THREAD_SAFETY to CaloClusterCorrection

  • static --> static const
  • Remove const in one place when is not needed
  • More interesting a couple of tools have initialize that potentially can register callbacks. Given the convention for MT initialize at large does not need to be thread safe. So Marked the specific methods as NOT_THREAD_SAFE. But as this is only potential (depends on job options) I did not mark the full class.
Edited Jan 25, 2020 by Christos Anastopoulos
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: CaloClusterCorrection_Check_thread