code-checks fix

cms-sw · Jul 14, 2021 · a1e3263 · a1e3263
1 parent 61a6883
commit a1e3263
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 14 deletions.
diff --git a/CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h b/CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h
@@ -13,7 +13,7 @@
 
 class L1TMuonBarrelParamsRcd : public edm::eventsetup::EventSetupRecordImplementation<L1TMuonBarrelParamsRcd> {};
 
-class L1TMuonBarrelKalmanParamsRcd :
-public edm::eventsetup::EventSetupRecordImplementation<L1TMuonBarrelKalmanParamsRcd> {};
+class L1TMuonBarrelKalmanParamsRcd
+    : public edm::eventsetup::EventSetupRecordImplementation<L1TMuonBarrelKalmanParamsRcd> {};
 
 #endif
diff --git a/CondFormats/L1TObjects/src/L1TMuonBarrelKalmanParams.cc b/CondFormats/L1TObjects/src/L1TMuonBarrelKalmanParams.cc
@@ -1,11 +1,11 @@
 #include "CondFormats/L1TObjects/interface/L1TMuonBarrelKalmanParams.h"
 
 L1TMuonBarrelKalmanParams::L1TMuonBarrelKalmanParams()
-  : pnodes_(NUM_BMTF_PARAM_NODES)//, l1mudttfmasks_(1)
+    : pnodes_(NUM_BMTF_PARAM_NODES)  //, l1mudttfmasks_(1)
 {
   version_ = Version;
   pnodes_[CONFIG].type_ = "unspecified";
-  pnodes_[CONFIG].fwVersion_ = 0; //default to recognize a RCD that is not filled
+  pnodes_[CONFIG].fwVersion_ = 0;  //default to recognize a RCD that is not filled
   // pnodes_[CONFIG].sparams_.clear();
   // pnodes_[CONFIG].iparams_.resize(NUM_CONFIG_PARAMS);
 }
diff --git a/L1Trigger/L1TMuonBarrel/plugins/L1TMuonBarrelKalmanParamsESProducer.cc b/L1Trigger/L1TMuonBarrel/plugins/L1TMuonBarrelKalmanParamsESProducer.cc
@@ -17,8 +17,8 @@
 #include "FWCore/Framework/interface/ESHandle.h"
 #include "FWCore/Framework/interface/ESProducts.h"
 
-#include "CondFormats/L1TObjects/interface/L1TMuonBarrelKalmanParams.h" //changed
-#include "CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h" //changed
+#include "CondFormats/L1TObjects/interface/L1TMuonBarrelKalmanParams.h"
+#include "CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h"
 #include "L1Trigger/L1TMuon/interface/MicroGMTLUTFactories.h"
 
 #include "FWCore/ParameterSet/interface/FileInPath.h"
@@ -28,7 +28,7 @@
 #include "L1Trigger/L1TCommon/interface/Parameter.h"
 #include "L1Trigger/L1TCommon/interface/Mask.h"
 
-#include "L1Trigger/L1TMuonBarrel/interface/L1TMuonBarrelParamsHelper.h" //to check
+#include "L1Trigger/L1TMuonBarrel/interface/L1TMuonBarrelParamsHelper.h"
 
 // class declaration
 //
@@ -59,7 +59,7 @@ class L1TMuonBarrelKalmanParamsESProducer : public edm::ESProducer {
 //
 L1TMuonBarrelKalmanParamsESProducer::L1TMuonBarrelKalmanParamsESProducer(const edm::ParameterSet& iConfig) {
   // the following line is needed to tell the framework what data is being produced
-  setWhatProduced(this); //???
+  setWhatProduced(this);
 
   // basic configurables needed (now set static)
   kalman_params.pnodes_[kalman_params.CONFIG].fwVersion_ = iConfig.getParameter<unsigned>("fwVersion");
@@ -75,7 +75,8 @@ L1TMuonBarrelKalmanParamsESProducer::~L1TMuonBarrelKalmanParamsESProducer() {}
 //
 
 // ------------ method called to produce the data  ------------
-L1TMuonBarrelKalmanParamsESProducer::ReturnType L1TMuonBarrelKalmanParamsESProducer::produce(const L1TMuonBarrelKalmanParamsRcd& iRecord) {
+L1TMuonBarrelKalmanParamsESProducer::ReturnType L1TMuonBarrelKalmanParamsESProducer::produce(
+    const L1TMuonBarrelKalmanParamsRcd& iRecord) {
   return std::make_unique<L1TMuonBarrelKalmanParams>(kalman_params);
 }
 

diff --git a/L1TriggerConfig/Utilities/src/L1TMuonBarrelKalmanParamsViewer.cc b/L1TriggerConfig/Utilities/src/L1TMuonBarrelKalmanParamsViewer.cc
@@ -25,7 +25,7 @@ class L1TMuonBarrelKalmanParamsViewer : public edm::EDAnalyzer {
 public:
   void analyze(const edm::Event &, const edm::EventSetup &) override;
 
-  L1TMuonBarrelKalmanParamsViewer(const edm::ParameterSet &) {};
+  L1TMuonBarrelKalmanParamsViewer(const edm::ParameterSet &){};
   ~L1TMuonBarrelKalmanParamsViewer(void) override {}
 };
 
@@ -64,7 +64,7 @@ void L1TMuonBarrelKalmanParamsViewer::analyze(const edm::Event &iEvent, const ed
   // Get the nodes and print out
   auto pnodes = ptr->pnodes_[ptr->CONFIG];
   cout << "version    : " << ptr->version_ << endl;
-  cout << "fwVersion : " << hex << pnodes.fwVersion_ << dec << endl;
+  cout << "fwVersion  : " << hex << pnodes.fwVersion_ << dec << endl;
   cout << "LUTsPath   : " << pnodes.kalmanLUTsPath_ << endl;
 
   // typedef std::map<short, short, std::less<short> > LUT;

diff --git a/L1TriggerConfig/Utilities/src/L1TMuonBarrelParamsWriter.cc b/L1TriggerConfig/Utilities/src/L1TMuonBarrelParamsWriter.cc
@@ -9,7 +9,7 @@
 #include "FWCore/Framework/interface/ESHandle.h"
 
 #include "CondFormats/DataRecord/interface/L1TMuonBarrelParamsO2ORcd.h"
-#include "CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h" // includes both Old-Phase1 and Kalman Rcds
+#include "CondFormats/DataRecord/interface/L1TMuonBarrelParamsRcd.h"  // includes both Old-Phase1 and Kalman Rcds
 #include "CondFormats/L1TObjects/interface/L1TMuonBarrelParams.h"
 #include "CondFormats/L1TObjects/interface/L1TMuonBarrelKalmanParams.h"
 
@@ -46,9 +46,10 @@ void L1TMuonBarrelParamsWriter::analyze(const edm::Event& iEvent, const edm::Eve
   edm::Service<cond::service::PoolDBOutputService> poolDb;
   if (poolDb.isAvailable()) {
     cond::Time_t firstSinceTime = poolDb->beginOfTime();
-    poolDb->writeOne( ptr1.get(), firstSinceTime, (isO2Opayload ? "L1TMuonBarrelParamsO2ORcd" : "L1TMuonBarrelParamsRcd") );
+    poolDb->writeOne(
+        ptr1.get(), firstSinceTime, (isO2Opayload ? "L1TMuonBarrelParamsO2ORcd" : "L1TMuonBarrelParamsRcd"));
     if (not isO2Opayload)
-      poolDb->writeOne( ptr2.get(), firstSinceTime, ("L1TMuonBarrelKalmanParamsRcd") );
+      poolDb->writeOne(ptr2.get(), firstSinceTime, ("L1TMuonBarrelKalmanParamsRcd"));
   }
 }