Add missing defaults in CalorimeterHit.h

LDMX-Software · Sep 4, 2024 · 39967f4 · 39967f4
1 parent 30e0b80
commit 39967f4
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 24 deletions.
diff --git a/Recon/include/Recon/Event/CalorimeterHit.h b/Recon/include/Recon/Event/CalorimeterHit.h
@@ -32,7 +32,7 @@ class CalorimeterHit {
   /**
    * Class destructor.
    */
-  virtual ~CalorimeterHit() {}
+  virtual ~CalorimeterHit() = default;
 
   /**
    * Clear the data in the object.
@@ -167,21 +167,21 @@ class CalorimeterHit {
   float time_{0};
 
   /** X Position of the hit */
-  float xpos_;
+  float xpos_{-9999.};
 
   /** Y Position of the hit */
-  float ypos_;
+  float ypos_{-9999.};
 
   /** Z Position of the hit */
-  float zpos_;
+  float zpos_{-9999.};
 
   /** Is this a noise hit? */
-  bool isNoise_;
+  bool isNoise_{false};
 
   /**
    * The ROOT class definition.
    */
-  ClassDef(CalorimeterHit, 1);
+  ClassDef(CalorimeterHit, 2);
 };
 }  // namespace ldmx
 

diff --git a/Recon/src/Recon/Event/CalorimeterHit.cxx b/Recon/src/Recon/Event/CalorimeterHit.cxx
@@ -3,24 +3,24 @@
 // STL
 #include <iostream>
 
-ClassImp(ldmx::CalorimeterHit)
+ClassImp(ldmx::CalorimeterHit);
 
-    namespace ldmx {
-  void CalorimeterHit::Clear() {
-    id_ = 0;
-    amplitude_ = 0;
-    energy_ = 0;
-    time_ = 0;
-    xpos_ = 0;
-    ypos_ = 0;
-    zpos_ = 0;
-    isNoise_ = false;
-  }
+namespace ldmx {
+void CalorimeterHit::Clear() {
+  id_ = 0;
+  amplitude_ = 0;
+  energy_ = 0;
+  time_ = 0;
+  xpos_ = 0;
+  ypos_ = 0;
+  zpos_ = 0;
+  isNoise_ = false;
+}
 
-  void CalorimeterHit::Print() const {
-    std::cout << "CalorimeterHit { "
-              << "id: " << std::hex << id_ << std::dec
-              << ",  energy: " << energy_ << "MeV, time: " << time_
-              << "ns, amplitude: " << amplitude_ << "}" << std::endl;
-  }
+void CalorimeterHit::Print() const {
+  std::cout << "CalorimeterHit { "
+            << "id: " << std::hex << id_ << std::dec << ",  energy: " << energy_
+            << "MeV, time: " << time_ << "ns, amplitude: " << amplitude_ << "}"
+            << std::endl;
+}
 }  // namespace ldmx