Remove using namespace std from muon packages (2)
Hi,
this MR removes unnecessary using namespace std
occurences from several muon packages. Furthermore, commented lines are removed and cout/cerr
are replaced by proper messaging.
Best, Nico
Merge request reports
Activity
This merge request affects 14 packages:
- MuonSpectrometer/MuonCalib/CscCalib/CscCalibAlgs
- MuonSpectrometer/MuonCalib/CscCalib/CscCalibData
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibData
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibDbAsciiTool
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibFitters
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibRt
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibT0
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibTools
- MuonSpectrometer/MuonCalib/MuonCalibExtraTree/MuonCalibExtraTreeAlg
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/CalibNtupleAnalysisAlg
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneBase
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneExtraTools
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneTools
- MuonSpectrometer/MuonCalib/MuonCalibUtils/MuonCalibMath
added MuonSpectrometer master review-pending-level-1 labels
CI Result FAILURE (hash 062278da)Athena AthSimulation AnalysisBase AthGeneration externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 1, 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 12346]added 16 commits
-
062278da...25cccb54 - 14 commits from branch
atlas:master
- e1a03575 - fix compilation of MuonCalibMath
- f60cf1c2 - Merge branch 'master' of ssh://gitlab.cern.ch:7999/atlas/athena into muNameSpc2
-
062278da...25cccb54 - 14 commits from branch
This merge request affects 14 packages:
- MuonSpectrometer/MuonCalib/CscCalib/CscCalibAlgs
- MuonSpectrometer/MuonCalib/CscCalib/CscCalibData
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibData
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibDbAsciiTool
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibFitters
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibRt
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibT0
- MuonSpectrometer/MuonCalib/MdtCalib/MdtCalibTools
- MuonSpectrometer/MuonCalib/MuonCalibExtraTree/MuonCalibExtraTreeAlg
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/CalibNtupleAnalysisAlg
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneBase
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneExtraTools
- MuonSpectrometer/MuonCalib/MuonCalibStandAlone/MuonCalibStandAloneTools
- MuonSpectrometer/MuonCalib/MuonCalibUtils/MuonCalibMath
CI Result SUCCESS (hash f60cf1c2)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 12400]added review-approved label and removed review-pending-level-1 label
- Resolved by Clara Nellist
@cnellist your comment said it was going to L2 but you set the approved label. Can you confirm which you meant.
Thanks
added review-pending-level-2 label and removed review-approved label
added review-approved label and removed review-pending-level-2 label
mentioned in commit 74dfe2ff
added sweep:ignore label