Skip to content

Commit 0cb0985

Browse files
missirolmmusich
authored andcommitted
improve const correctness in PhotonXGBoostProducer
1 parent 7a45a69 commit 0cb0985

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ class PhotonXGBoostProducer : public edm::global::EDProducer<> {
3838
const unsigned mvaNTreeLimitB_;
3939
const unsigned mvaNTreeLimitE_;
4040
const double mvaThresholdEt_;
41-
std::unique_ptr<PhotonXGBoostEstimator> mvaEstimatorB_;
42-
std::unique_ptr<PhotonXGBoostEstimator> mvaEstimatorE_;
41+
const std::unique_ptr<const PhotonXGBoostEstimator> mvaEstimatorB_;
42+
const std::unique_ptr<const PhotonXGBoostEstimator> mvaEstimatorE_;
4343
};
4444

4545
PhotonXGBoostProducer::PhotonXGBoostProducer(edm::ParameterSet const& config)
@@ -54,9 +54,9 @@ PhotonXGBoostProducer::PhotonXGBoostProducer(edm::ParameterSet const& config)
5454
mvaFileXgbE_(config.getParameter<edm::FileInPath>("mvaFileXgbE")),
5555
mvaNTreeLimitB_(config.getParameter<unsigned int>("mvaNTreeLimitB")),
5656
mvaNTreeLimitE_(config.getParameter<unsigned int>("mvaNTreeLimitE")),
57-
mvaThresholdEt_(config.getParameter<double>("mvaThresholdEt")) {
58-
mvaEstimatorB_ = std::make_unique<PhotonXGBoostEstimator>(mvaFileXgbB_, mvaNTreeLimitB_);
59-
mvaEstimatorE_ = std::make_unique<PhotonXGBoostEstimator>(mvaFileXgbE_, mvaNTreeLimitE_);
57+
mvaThresholdEt_(config.getParameter<double>("mvaThresholdEt")),
58+
mvaEstimatorB_{std::make_unique<const PhotonXGBoostEstimator>(mvaFileXgbB_, mvaNTreeLimitB_)},
59+
mvaEstimatorE_{std::make_unique<const PhotonXGBoostEstimator>(mvaFileXgbE_, mvaNTreeLimitE_)} {
6060
produces<reco::RecoEcalCandidateIsolationMap>();
6161
}
6262

0 commit comments

Comments
 (0)