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 46
    • Merge requests 46
  • 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
  • !3189

Modernize UTDet

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Gerhard Raven requested to merge modernize-utdet into master Aug 15, 2021
  • Overview 18
  • Commits 3
  • Pipelines 6
  • Changes 21
  • prefer pass-by-value for small objects
  • remove redundant dynamic_cast
  • drop redundant const qualifiers
  • prefer [[maybe_unused]] over #ifdef NDEBUG
  • include the relevant header
  • move DeUTSector::Status conversion to/from string to DeUTSector
  • add functions needed to support Gaudi::PropertyDeUTSector::Status
  • move the functions in UTDetFun to the .cpp files which use them, and retire UTDetFun.h
  • make the member functions of UTNames static

must be applied in conjunction with Lbcom!575 (merged)

Edited Aug 16, 2021 by Gerhard Raven
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: modernize-utdet