[VISUALIZATION] Changes suggested by new llvm18 clang-format

Fireworks/Calo/interface/scaleMarker.h

@@ -24,7 +24,7 @@ class FWViewContext;
 namespace fireworks {
   struct scaleMarker {
     scaleMarker(TEveScalableStraightLineSet* ls, float et, float e, const FWViewContext* vc)
-        : m_ls(ls), m_et(et), m_energy(e), m_vc(vc){};
+        : m_ls(ls), m_et(et), m_energy(e), m_vc(vc) {}
 
     virtual ~scaleMarker() {}
 

Fireworks/Calo/plugins/FWTracksterHitsProxyBuilder.cc

@@ -262,7 +262,7 @@ void FWTracksterHitsProxyBuilder::build(const ticl::Trackster &iData,
         }
       }
     }  // End of loop over rechits of a single layercluster
-  }    // End loop over the layerclusters of the trackster
+  }  // End loop over the layerclusters of the trackster
 
   hex_boxset->RefitPlex();
   boxset->RefitPlex();

Fireworks/Core/interface/FWJobMetadataUpdateRequest.h

@@ -3,7 +3,7 @@
 
 class FWJobMetadataUpdateRequest {
 public:
-  virtual ~FWJobMetadataUpdateRequest(){};
+  virtual ~FWJobMetadataUpdateRequest() {}
 };
 
 #endif

Fireworks/Core/interface/FWProxyBuilderBase.h

@@ -110,7 +110,7 @@ class FWProxyBuilderBase {
 
   virtual void modelChanges(const FWModelIds&, Product*);
 
-  virtual void scaleProduct(TEveElementList* parent, FWViewType::EType, const FWViewContext* vc){};
+  virtual void scaleProduct(TEveElementList* parent, FWViewType::EType, const FWViewContext* vc) {}
 
   FWProxyBuilderBase(const FWProxyBuilderBase&);                   // stop default
   const FWProxyBuilderBase& operator=(const FWProxyBuilderBase&);  // stop default

Fireworks/Core/interface/FWViewManagerBase.h

@@ -61,8 +61,8 @@ class FWViewManagerBase {
   void modelChangesComingSlot();
   void modelChangesDoneSlot();
   void colorsChangedSlot();
-  virtual void eventBegin(){};
-  virtual void eventEnd(){};
+  virtual void eventBegin() {}
+  virtual void eventEnd() {}
 
   // ---------- const member functions ---------------------
 

Fireworks/Core/src/CSGConnector.h

@@ -29,7 +29,7 @@ class CSGActionSupervisor;
 
 class CSGConnector : public TQObject {
 public:
-  CSGConnector(CSGAction *action, CSGActionSupervisor *supervisor) : m_action(action), m_supervisor(supervisor){};
+  CSGConnector(CSGAction *action, CSGActionSupervisor *supervisor) : m_action(action), m_supervisor(supervisor) {}
   //virtual ~CSGConnector();
 
   // ---------- member functions ---------------------------

Fireworks/Core/src/FW3DViewDistanceMeasureTool.h

@@ -14,7 +14,7 @@ class FW3DViewDistanceMeasureTool {
   enum EPickAction { kPnt1, kPnt2, kNone };
 
   FW3DViewDistanceMeasureTool();
-  virtual ~FW3DViewDistanceMeasureTool(){};
+  virtual ~FW3DViewDistanceMeasureTool() {}
 
   void resetAction();
 

Fireworks/Core/src/FWGeometryTableViewBase.cc

@@ -114,7 +114,7 @@ namespace {
                                  kStructureNotifyMask | kFocusChangeMask | kEnterWindowMask | kLeaveWindowMask);
     }
 
-    ~FWGeometryVF() override{};
+    ~FWGeometryVF() override {}
 
     Bool_t HandleKey(Event_t* event) override {
       if (event->fCode == (UInt_t)gVirtualX->KeysymToKeycode(kKey_Escape)) {

Fireworks/Core/src/fwPaletteExtra.cc

@@ -13,7 +13,7 @@ namespace fireworks {
     bool whiteBg;
     FWColorManager::EPalette id;
     PaletteExtraTopBottomIndices(float (*iColors)[3], bool iIsWhiteBg, FWColorManager::EPalette iId)
-        : colVals(iColors), whiteBg(iIsWhiteBg), id(iId){};
+        : colVals(iColors), whiteBg(iIsWhiteBg), id(iId) {}
 
     void set(int i, Color_t cidx) {
       if (whiteBg) {

Fireworks/FWInterface/src/FWPSetCellEditor.h

@@ -23,8 +23,8 @@
 
 class FWPSetCellEditor : public TGTextEntry {
 public:
-  FWPSetCellEditor(const TGWindow* w, const char* txt) : TGTextEntry(w, txt){};
-  ~FWPSetCellEditor() override{};
+  FWPSetCellEditor(const TGWindow* w, const char* txt) : TGTextEntry(w, txt) {}
+  ~FWPSetCellEditor() override {}
   bool HandleKey(Event_t* event) override;
   bool apply(FWPSetTableManager::PSetData& data, FWPSetTableManager::PSetData& parent);
 };

Fireworks/Tracks/plugins/FWItemTrackAccessors.cc

@@ -29,7 +29,7 @@
 class BeamSpotSingleAccessor : public FWItemSingleAccessor {
 public:
   BeamSpotSingleAccessor(const TClass* x) : FWItemSingleAccessor(x) {}
-  ~BeamSpotSingleAccessor() override{};
+  ~BeamSpotSingleAccessor() override {}
   REGISTER_FWITEMACCESSOR_METHODS();
 };
 

Fireworks/Tracks/plugins/FWSiPixelDigiProxyBuilder.cc

@@ -71,7 +71,7 @@ void FWSiPixelDigiProxyBuilder::build(const FWEventItem* iItem, TEveElementList*
         pointSet->SetNextPoint(globalPoint[0], globalPoint[1], globalPoint[2]);
       }
     }  // end of iteration over digis in range
-  }    // end of iteration over the DetSetVector
+  }  // end of iteration over the DetSetVector
 }
 
 REGISTER_FWPROXYBUILDER(FWSiPixelDigiProxyBuilder,

Fireworks/Tracks/plugins/FWSiStripDigiProxyBuilder.cc

@@ -72,7 +72,7 @@ void FWSiStripDigiProxyBuilder::build(const FWEventItem* iItem, TEveElementList*
 
       lineSet->AddLine(globalTop[0], globalTop[1], globalTop[2], globalBottom[0], globalBottom[1], globalBottom[2]);
     }  // end of iteration over digis
-  }    // end of iteration over the DetSetVector
+  }  // end of iteration over the DetSetVector
 }
 
 REGISTER_FWPROXYBUILDER(FWSiStripDigiProxyBuilder,