From 407f38cab5afb10dc6af27c73a465e04903feab3 Mon Sep 17 00:00:00 2001 From: Michal Bluj Date: Tue, 19 May 2020 17:35:44 +0200 Subject: [PATCH] code-checks fixes --- .../TauDiscriminationAgainstElectronDeadECAL.cc | 6 ++++-- .../RecoTau/src/TauDiscriminationProducerBase.cc | 16 +++++++++------- 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/RecoTauTag/RecoTau/plugins/TauDiscriminationAgainstElectronDeadECAL.cc b/RecoTauTag/RecoTau/plugins/TauDiscriminationAgainstElectronDeadECAL.cc index 4a94dc614ca58..2905fb0401124 100644 --- a/RecoTauTag/RecoTau/plugins/TauDiscriminationAgainstElectronDeadECAL.cc +++ b/RecoTauTag/RecoTau/plugins/TauDiscriminationAgainstElectronDeadECAL.cc @@ -37,8 +37,10 @@ class TauDiscriminationAgainstElectronDeadECAL : public TauDiscriminationProduce double discriminate(const TauRef& tau) const override { if (verbosity_) { edm::LogPrint(TauDiscriminationProducerBase::getTauTypeString() + "AgainstEleDeadECAL") - << "<" + TauDiscriminationProducerBase::getTauTypeString() + "AgainstElectronDeadECAL::discriminate>:"; - edm::LogPrint(TauDiscriminationProducerBase::getTauTypeString() + "AgainstEleDeadECAL") << " moduleLabel = " << moduleLabel_; + << "<" + TauDiscriminationProducerBase::getTauTypeString() + + "AgainstElectronDeadECAL::discriminate>:"; + edm::LogPrint(TauDiscriminationProducerBase::getTauTypeString() + "AgainstEleDeadECAL") + << " moduleLabel = " << moduleLabel_; edm::LogPrint(TauDiscriminationProducerBase::getTauTypeString() + "AgainstEleDeadECAL") << " tau: Pt = " << tau->pt() << ", eta = " << tau->eta() << ", phi = " << tau->phi(); } diff --git a/RecoTauTag/RecoTau/src/TauDiscriminationProducerBase.cc b/RecoTauTag/RecoTau/src/TauDiscriminationProducerBase.cc index a21fe26fc838c..50d25e568e6cd 100644 --- a/RecoTauTag/RecoTau/src/TauDiscriminationProducerBase.cc +++ b/RecoTauTag/RecoTau/src/TauDiscriminationProducerBase.cc @@ -40,7 +40,7 @@ TauDiscriminationProducerBase("@module_label")) { // tau collection to discriminate - TauProducer_ = iConfig.getParameter(getTauTypeString()+"Producer"); + TauProducer_ = iConfig.getParameter(getTauTypeString() + "Producer"); Tau_token = consumes(TauProducer_); // prediscriminant operator @@ -172,7 +172,7 @@ template :: fillProducerDescriptions(edm::ParameterSetDescription& desc) { // helper function, it fills the description of the Producers parameter - desc.add(getTauTypeString()+"Producer", edm::InputTag("fixme")); + desc.add(getTauTypeString() + "Producer", edm::InputTag("fixme")); { edm::ParameterSetDescription pset_prediscriminants; pset_prediscriminants.add("BooleanOperator", "AND"); @@ -192,12 +192,14 @@ void TauDiscriminationProducerBase -std::string TauDiscriminationProducerBase:: - getTauTypeString() { - if(std::is_same::value) return "PFTau"; - if(std::is_same::value) return "PATTau"; +std::string +TauDiscriminationProducerBase::getTauTypeString() { + if (std::is_same::value) + return "PFTau"; + if (std::is_same::value) + return "PATTau"; throw cms::Exception("TauDiscriminationProducerBase") - << "Unsupported TauType used. You must use either PFTau or PATTau."; + << "Unsupported TauType used. You must use either PFTau or PATTau."; } // compile our desired types and make available to linker