MuonGeoModel remove ref/unref calls
The PVLinks manage the reference counting by themselves now. No need to call ref & unref again.
Merge request reports
Activity
This merge request affects 1 package:
- MuonSpectrometer/MuonGeoModel
This merge request affects 5 files:
- MuonSpectrometer/MuonGeoModel/src/Cutout.cxx
- MuonSpectrometer/MuonGeoModel/src/Ded.cxx
- MuonSpectrometer/MuonGeoModel/src/MultiLayer.cxx
- MuonSpectrometer/MuonGeoModel/src/MuonChamber.cxx
- MuonSpectrometer/MuonGeoModel/src/RpcLayer.cxx
Adding @apsallid ,@kluit ,@jojungge ,@pscholer ,@rosati ,@stavrop as watchers
added MuonSpectrometer main review-pending-level-1 labels
added full-integration-tests full-unit-tests labels
CI Result SUCCESS (hash be9e7082)Athena AthSimulation externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4324]The changes look okay and there are no issues with the CI; the externals warning is from ATLINFR-5244. Approving from L1.
Edited by James Beachamadded review-approved label and removed review-pending-level-1 label
This merge request affects 1 package:
- MuonSpectrometer/MuonGeoModel
This merge request affects 5 files:
- MuonSpectrometer/MuonGeoModel/src/Cutout.cxx
- MuonSpectrometer/MuonGeoModel/src/Ded.cxx
- MuonSpectrometer/MuonGeoModel/src/MultiLayer.cxx
- MuonSpectrometer/MuonGeoModel/src/MuonChamber.cxx
- MuonSpectrometer/MuonGeoModel/src/RpcLayer.cxx
Adding @stavrop ,@kluit ,@jojungge ,@rosati ,@apsallid ,@pscholer as watchers
added review-pending-level-1 label and removed review-approved label
CI Result SUCCESS (hash be9e7082)Athena AthSimulation externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4340]added review-approved label and removed review-pending-level-1 label
mentioned in commit 929977b1