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 172
    • Merge requests 172
  • 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
  • !23106

Merged
Created May 01, 2019 by Christos Anastopoulos@christosDeveloper

static --> static const for GeometryStatics

  • Overview 10
  • Commits 2
  • Pipelines 2
  • Changes 1

This should fix warnings like this one

In file included from /build1/atnight/localbuilds/nightlies/master/athena/Tracking/TrkFitter/TrkGaussianSumFilter/src/GaussianSumFitter.cxx:21:
/build1/atnight/localbuilds/nightlies/master/athena/Tracking/TrkEvent/TrkEventUtils/TrkEventUtils/MeasurementBaseComparisonFunction.h: In member function 'double Trk::MeasurementBaseComparisonFunction::pathIntersectWithLine(const Trk::PerigeeSurface&) const':
/build1/atnight/localbuilds/nightlies/master/athena/Tracking/TrkEvent/TrkEventUtils/TrkEventUtils/MeasurementBaseComparisonFunction.h:191:48: warning: Static expression 'Trk::s_zAxis' of type 'Amg::Vector3D {aka Eigen::Matrix<double, 3, 1>}' passed to pointer or reference function argument of 'Eigen::MatrixBase<Eigen::Matrix<double, 3, 1> >::dot<Eigen::Matrix<double, 3, 1> >' within function 'double Trk::MeasurementBaseComparisonFunction::pathIntersectWithLine(const Trk::PerigeeSurface&) const'; may not be thread-safe.
       double     parallelity = m_direction->dot(Trk::s_zAxis);
                                ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~

they started appearing after we added the static thread checker in a couple of packages. So I guess we catched a potential issue or so.

Mentioning @ssnyder and @oda that might have also seen these

Edited May 02, 2019 by Christos Anastopoulos
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: GeometryStatics_const