Code check

Author: Sunanda

Geometry/CaloTopology/test/CaloTowerMapTester.cc

@@ -42,7 +42,9 @@ class CaloTowerMapTester : public edm::one::EDAnalyzer<edm::one::WatchRuns> {
   const edm::ESGetToken<CaloTowerConstituentsMap, CaloGeometryRecord> tokMap_;
 };
 
-CaloTowerMapTester::CaloTowerMapTester(const edm::ParameterSet&) : tokGeom_{esConsumes<CaloGeometry, CaloGeometryRecord>(edm::ESInputTag{})}, tokMap_{esConsumes<CaloTowerConstituentsMap, CaloGeometryRecord>(edm::ESInputTag{})} {}
+CaloTowerMapTester::CaloTowerMapTester(const edm::ParameterSet&)
+    : tokGeom_{esConsumes<CaloGeometry, CaloGeometryRecord>(edm::ESInputTag{})},
+      tokMap_{esConsumes<CaloTowerConstituentsMap, CaloGeometryRecord>(edm::ESInputTag{})} {}
 
 void CaloTowerMapTester::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
   edm::ParameterSetDescription desc;
@@ -63,7 +65,8 @@ void CaloTowerMapTester::doTest(const CaloGeometry* geo, const CaloTowerConstitu
   for (const auto& id : dets) {
     CaloTowerDetId tower = ctmap->towerOf(id);
     std::vector<DetId> ids = ctmap->constituentsOf(tower);
-    std::cout << static_cast<HcalDetId>(id) << " belongs to " << tower << " which has " << ids.size() << " constituents\n";
+    std::cout << static_cast<HcalDetId>(id) << " belongs to " << tower << " which has " << ids.size()
+              << " constituents\n";
     for (unsigned int i = 0; i < ids.size(); ++i) {
       std::cout << "[" << i << "] " << std::hex << ids[i].rawId() << std::dec;
       if (ids[i].det() == DetId::Ecal && ids[i].subdetId() == EcalBarrel) {

Geometry/CaloTopology/test/CaloTowerTopologyTester.cc

@@ -36,7 +36,8 @@ class CaloTowerTopologyTester : public edm::one::EDAnalyzer<edm::one::WatchRuns>
   const edm::ESGetToken<CaloTowerTopology, HcalRecNumberingRecord> tokTopo_;
 };
 
-CaloTowerTopologyTester::CaloTowerTopologyTester(const edm::ParameterSet&) : tokTopo_{esConsumes<CaloTowerTopology, HcalRecNumberingRecord>(edm::ESInputTag{})} {}
+CaloTowerTopologyTester::CaloTowerTopologyTester(const edm::ParameterSet&)
+    : tokTopo_{esConsumes<CaloTowerTopology, HcalRecNumberingRecord>(edm::ESInputTag{})} {}
 
 void CaloTowerTopologyTester::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
   edm::ParameterSetDescription desc;

Geometry/CaloTopology/test/FastTimeTopologyTester.cc

@@ -34,7 +34,8 @@ class FastTimeTopologyTester : public edm::one::EDAnalyzer<edm::one::WatchRuns>
   const edm::ESGetToken<FastTimeTopology, IdealGeometryRecord> tokTopo_;
 };
 
-FastTimeTopologyTester::FastTimeTopologyTester(const edm::ParameterSet&) : tokTopo_{esConsumes<FastTimeTopology, IdealGeometryRecord>(edm::ESInputTag{"", "FastTimeBarrel"})} {}
+FastTimeTopologyTester::FastTimeTopologyTester(const edm::ParameterSet&)
+    : tokTopo_{esConsumes<FastTimeTopology, IdealGeometryRecord>(edm::ESInputTag{"", "FastTimeBarrel"})} {}
 
 void FastTimeTopologyTester::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
   edm::ParameterSetDescription desc;

Geometry/CaloTopology/test/HGCalTopologyTester.cc

@@ -45,7 +45,7 @@ HGCalTopologyTester::HGCalTopologyTester(const edm::ParameterSet& iC)
       sec2_(iC.getParameter<std::vector<int> >("sector2")),
       cell1_(iC.getParameter<std::vector<int> >("cell1")),
       cell2_(iC.getParameter<std::vector<int> >("cell2")),
-      tokTopo_{esConsumes<HGCalTopology, IdealGeometryRecord>(edm::ESInputTag{"", detectorName_ })} {}
+      tokTopo_{esConsumes<HGCalTopology, IdealGeometryRecord>(edm::ESInputTag{"", detectorName_})} {}
 
 void HGCalTopologyTester::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
   edm::ParameterSetDescription desc;

Geometry/CaloTopology/test/HcalTopologyTester.cc

@@ -34,17 +34,16 @@ class HcalTopologyTester : public edm::one::EDAnalyzer<edm::one::WatchRuns> {
   const edm::ESGetToken<HcalTopology, HcalRecNumberingRecord> tokTopo_;
 };
 
-HcalTopologyTester::HcalTopologyTester(const edm::ParameterSet&) : tokTopo_{esConsumes<HcalTopology, HcalRecNumberingRecord>(edm::ESInputTag{})} {}
+HcalTopologyTester::HcalTopologyTester(const edm::ParameterSet&)
+    : tokTopo_{esConsumes<HcalTopology, HcalRecNumberingRecord>(edm::ESInputTag{})} {}
 
 void HcalTopologyTester::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
   edm::ParameterSetDescription desc;
   desc.setUnknown();
   descriptions.addDefault(desc);
 }
 
-void HcalTopologyTester::analyze(edm::Event const&, edm::EventSetup const& iSetup) {
-  doTest(iSetup.getData(tokTopo_));
-}
+void HcalTopologyTester::analyze(edm::Event const&, edm::EventSetup const& iSetup) { doTest(iSetup.getData(tokTopo_)); }
 
 void HcalTopologyTester::doTest(const HcalTopology& topology) {
   // First test on movements along eta/phi directions