Skip to content
Snippets Groups Projects

use atomic variables to replace the mutable variables

Merged Ke Li requested to merge keli/athena:CSCcablingSvc into master

Use the thread-safe atomic for m_side, m_rod and m_max to replace the mutable variables in CSCcabling.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Nicolas Koehler
  • Ke Li added 1 commit

    added 1 commit

    • cf638c7c - remove tthe "mutable" for m_side, m_rod, m_max and m_run1

    Compare with previous version

  • Ke Li resolved all threads

    resolved all threads

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonCablings/CSCcabling

    Adding @rosati ,@nkoehler ,@wleight as watchers

  • CI Result FAILURE (hash 9978331f)

    Athena AthSimulation AnalysisBase AthGeneration
    externals
    cmake
    make
    required tests
    optional tests

    Full details available on this CI monitor view
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 12684]

  • CI Result FAILURE (hash cf638c7c)

    Athena AthSimulation AnalysisBase AthGeneration
    externals
    cmake
    make
    required tests
    optional tests

    Full details available on this CI monitor view
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 12690]

  • Looks like the DQ test timed out ...

    MLB (L1)

  • Jenkins please retry a build

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonCablings/CSCcabling

    Adding @rosati ,@nkoehler ,@wleight as watchers

  • CI Result FAILURE (hash cf638c7c)

    Athena AthSimulation AnalysisBase AthGeneration
    externals
    cmake
    make
    required tests
    optional tests

    Full details available on this CI monitor view
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 12755]

  • Jenkins please retry a build

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonCablings/CSCcabling

    Adding @rosati ,@nkoehler ,@wleight as watchers

  • CI Result FAILURE (hash cf638c7c)

    Athena AthSimulation AnalysisBase AthGeneration
    externals
    cmake
    make
    required tests
    optional tests

    Full details available on this CI monitor view
    Athena: number of compilation errors 0, warnings 0
    AthSimulation: number of compilation errors 0, warnings 0
    AnalysisBase: number of compilation errors 0, warnings 0
    AthGeneration: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 12789]

  • Failure is known. Changes look fine. Approving. Clara (L1)

  • added review-approved label and removed review-pending-level-1 label

  • mentioned in commit 99f0b537

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading