Minor clean of the constructors

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoAccessSurfaceAction.h

@@ -24,7 +24,7 @@ class GeoAccessSurfaceAction final : public GeoNodeAction
 {
  public:
   GeoAccessSurfaceAction (unsigned int Index, const GeoVAlignmentStore* store);
-  virtual ~GeoAccessSurfaceAction() override;
+  virtual ~GeoAccessSurfaceAction() = default;
 
   /// Handles a Transform.
   virtual void handleTransform (const GeoTransform *xform) override;
@@ -48,10 +48,6 @@ class GeoAccessSurfaceAction final : public GeoNodeAction
   const GeoTrf::Transform3D & getDefTransform () const;
 
  private:
-
-  GeoAccessSurfaceAction(const GeoAccessSurfaceAction &right);
-  GeoAccessSurfaceAction & operator=(const GeoAccessSurfaceAction &right);
-
   /// Returns a pointer to the ith virtual surface under this one.
   VSConstLink m_surface{};
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoAccessVolAndSTAction.h

@@ -46,9 +46,6 @@ class GeoAccessVolAndSTAction final : public GeoNodeAction {
   std::optional<unsigned int> getId() const;
 
  private:
-  GeoAccessVolAndSTAction(const GeoAccessVolAndSTAction &right);
-  const GeoAccessVolAndSTAction & operator=(const GeoAccessVolAndSTAction &right);
-
   /// A pointer to the ith physical volume under this one.
   PVConstLink m_volume{};
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoClearAbsPosAction.h

@@ -16,18 +16,15 @@
 
 #include "GeoModelKernel/GeoNodeAction.h"
 
-class GeoClearAbsPosAction : public GeoNodeAction
-{
+class GeoClearAbsPosAction : public GeoNodeAction {
  public:
-  GeoClearAbsPosAction();
-  virtual ~GeoClearAbsPosAction();
+  GeoClearAbsPosAction() = default;
+  virtual ~GeoClearAbsPosAction() = default;
 
   //	Handles a physical volume.
-  virtual void handleFullPhysVol (const GeoFullPhysVol *vol);
+  virtual void handleFullPhysVol (const GeoFullPhysVol *vol) override;
 
  private:
-  GeoClearAbsPosAction(const GeoClearAbsPosAction &right);
-  GeoClearAbsPosAction & operator=(const GeoClearAbsPosAction &right);
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoCountSurfaceAction.h

@@ -11,7 +11,7 @@ class GeoCountSurfaceAction final: public GeoNodeAction
 {
  public:
   GeoCountSurfaceAction ();
-  virtual ~GeoCountSurfaceAction();
+  virtual ~GeoCountSurfaceAction() = default;
 
   //	Handles a virtual surface.
   virtual void handleVSurface (const GeoVSurface *) override;

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoCountVolAction.h

@@ -11,7 +11,7 @@ class GeoCountVolAction : public GeoNodeAction
 {
  public:
   GeoCountVolAction ();
-  virtual ~GeoCountVolAction();
+  virtual ~GeoCountVolAction() = default;
 
   //	Handles a physical volume.
   virtual void handlePhysVol (const GeoPhysVol *);
@@ -29,10 +29,8 @@ class GeoCountVolAction : public GeoNodeAction
   unsigned int getCount () const;
 
  private:
-  GeoCountVolAction(const GeoCountVolAction &right);
-  GeoCountVolAction & operator=(const GeoCountVolAction &right);
 
-  unsigned int m_count;
+  unsigned int m_count{0};
 };
 
 inline unsigned int GeoCountVolAction::getCount () const

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoCountVolAndSTAction.h

@@ -18,7 +18,7 @@ class GeoCountVolAndSTAction : public GeoNodeAction
 {
  public:
   GeoCountVolAndSTAction();
-  virtual ~GeoCountVolAndSTAction();
+  virtual ~GeoCountVolAndSTAction() = default;
 
   virtual void handlePhysVol(const GeoPhysVol*);
   virtual void handleFullPhysVol(const GeoFullPhysVol*);
@@ -27,10 +27,7 @@ class GeoCountVolAndSTAction : public GeoNodeAction
   unsigned int getCount() const;
 
  private:
-  GeoCountVolAndSTAction(const GeoCountVolAndSTAction &right);
-  const GeoCountVolAndSTAction & operator=(const GeoCountVolAndSTAction &right);
-
-  unsigned int m_count;
+  unsigned int m_count{0};
 };
 
 inline unsigned int GeoCountVolAndSTAction::getCount() const

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoShapeAction.h

@@ -138,16 +138,16 @@ class GeoShapeAction
 
   
  private:
-  GeoShapeAction(const GeoShapeAction &right);
-  GeoShapeAction & operator=(const GeoShapeAction &right);
+  GeoShapeAction(const GeoShapeAction &right) = delete;
+  GeoShapeAction & operator=(const GeoShapeAction &right) = delete;
 
   //	A limit may be placed upon the depth to which the action
   //	descends.  0 = self.  1 = self and children.
-  std::optional<unsigned int> m_depth;
+  std::optional<unsigned int> m_depth{std::nullopt};
   
   //	Termination flag; causes an abortion of action execution.
-  bool m_terminate;
-  GeoShapePath m_path;
+  bool m_terminate{false};
+  GeoShapePath m_path{};
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoSurfaceCursor.h

@@ -48,21 +48,21 @@ class GeoSurfaceCursor final : public GeoNodeAction
   
   /// Ressucitate (undo terminate)
   void resuscitate();
-  
-  PVConstLink                           m_parent;
-  PVConstLink                           m_volume;
-  VSConstLink                           m_surface;
-  GeoTrf::Transform3D                   m_transform;
-  GeoTrf::Transform3D                   m_defTransform;
+ 
+  PVConstLink                           m_parent{nullptr};
+  PVConstLink                           m_volume{nullptr};
+  VSConstLink                           m_surface{nullptr};
+  GeoTrf::Transform3D                   m_transform{GeoTrf::Transform3D::Identity()};
+  GeoTrf::Transform3D                   m_defTransform{GeoTrf::Transform3D::Identity()};
       
-  unsigned int                          m_majorIndex;
-  unsigned int                          m_volCount;
-  unsigned int                          m_surfCount;
+  unsigned int                          m_majorIndex{0};
+  unsigned int                          m_volCount{0};
+  unsigned int                          m_surfCount{0};
    
-  std::vector<const GeoTransform *>     m_pendingTransformList;
+  std::vector<const GeoTransform *>     m_pendingTransformList{};
   
   bool                                  m_hasAlignTrans{};  
-  GeoVAlignmentStore                   *m_alignStore;
+  GeoVAlignmentStore                   *m_alignStore{nullptr};
 };
 
 #endif

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVDetectorFactory.h

@@ -22,8 +22,11 @@ class GeoVDetectorManager;
 class GeoVDetectorFactory 
 {
  public:
-  GeoVDetectorFactory();
-  virtual ~GeoVDetectorFactory();
+  GeoVDetectorFactory() = default;
+  virtual ~GeoVDetectorFactory() = default;
+
+  GeoVDetectorFactory(const GeoVDetectorFactory &right) = delete;
+  GeoVDetectorFactory & operator=(const GeoVDetectorFactory &right) = delete;
 
   //	Create the system.
   virtual void create (GeoPhysVol* world) = 0;
@@ -33,8 +36,6 @@ class GeoVDetectorFactory
   virtual const GeoVDetectorManager* getDetectorManager () const = 0;
 
  private:
-  GeoVDetectorFactory(const GeoVDetectorFactory &right);
-  GeoVDetectorFactory & operator=(const GeoVDetectorFactory &right);
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVDetectorManager.h

@@ -28,8 +28,11 @@ class GeoVDetectorElement;
 class GeoVDetectorManager 
 {
  public:
-  GeoVDetectorManager();
-  virtual ~GeoVDetectorManager();
+  GeoVDetectorManager() = default;
+  virtual ~GeoVDetectorManager() = default;
+
+  GeoVDetectorManager(const GeoVDetectorManager &right) = delete;
+  GeoVDetectorManager & operator=(const GeoVDetectorManager &right) = delete;
 
   //	Returns the number of Tree Tops (Top Level Physicsl
   //	Volumes)
@@ -42,10 +45,8 @@ class GeoVDetectorManager
   void                setName(const std::string & name);
 
  private:
-  GeoVDetectorManager(const GeoVDetectorManager &right);
-  GeoVDetectorManager & operator=(const GeoVDetectorManager &right);
 
-  std::string m_name;
+  std::string m_name{};
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVG4ExtensionSolid.h

@@ -19,9 +19,9 @@ class GeoVG4ExtensionSolid
  public:
 
    //! Default constructor.
-   GeoVG4ExtensionSolid() {}
+   GeoVG4ExtensionSolid()  = default;
      
-   virtual ~GeoVG4ExtensionSolid() {}
+   virtual ~GeoVG4ExtensionSolid() = default;
 
    //! Create the specified solid. .
    virtual G4VSolid *newG4Solid(const GeoUnidentifiedShape *shape) const=0;

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVGeometryPlugin.h

@@ -29,6 +29,9 @@ class GeoVGeometryPlugin
     GeoVGeometryPlugin(const std::string& name) : 
         m_pluginName{name} {  m_publisher->setName(m_pluginName); } 
     
+    /// We prohibit copy constructor, and assignment operator
+    GeoVGeometryPlugin(const GeoVGeometryPlugin &right)=delete;
+    GeoVGeometryPlugin & operator=(const GeoVGeometryPlugin &right)=delete;
 
     virtual ~GeoVGeometryPlugin()  = default;
 
@@ -48,14 +51,8 @@ class GeoVGeometryPlugin
   std::unique_ptr<GeoPublisher> m_publisher{std::make_unique<GeoPublisher>()};
   
  private:
-
-  /// We prohibit copy constructor, and assignment operator
-  GeoVGeometryPlugin(const GeoVGeometryPlugin &right)=delete;
-  GeoVGeometryPlugin & operator=(const GeoVGeometryPlugin &right)=delete;
-
-
   //! The name of the plugin, used to store plugin's published nodes. 
-  std::string m_pluginName;
+  std::string m_pluginName{};
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVolumeAction.h

@@ -26,7 +26,7 @@ class GeoVolumeAction
 
  public:
   GeoVolumeAction (Type type = TOP_DOWN);
-  virtual ~GeoVolumeAction();
+  virtual ~GeoVolumeAction() = default;
 
   //	Callback method. Overriden by users.
   virtual void handleVPhysVol (const GeoVPhysVol *);
@@ -50,17 +50,15 @@ class GeoVolumeAction
   GeoVolumeAction::Type getType () const;
   
  private:
-  GeoVolumeAction(const GeoVolumeAction &right);
-  GeoVolumeAction & operator=(const GeoVolumeAction &right);
   
   //	This type specifies that the action should be carried
   //	out from the bottom up or from the top down.
-  Type m_type;
+  Type m_type{Type::TOP_DOWN};
 
   //	Termination flag; causes an abortion of action execution.
-  bool m_terminate;
+  bool m_terminate{false};
 
-  GeoTraversalState m_traversalState;
+  GeoTraversalState m_traversalState{};
   
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVolumeCursor.h

@@ -17,10 +17,7 @@ class GeoVolumeCursor final : public GeoNodeAction
  public:
   using VSConstLink = GeoVSurface::VSConstLink;
   GeoVolumeCursor (PVConstLink parent, GeoVAlignmentStore* store=nullptr);
-  virtual ~GeoVolumeCursor() override;
- 
-  GeoVolumeCursor(const GeoVolumeCursor &right) = delete;
-  GeoVolumeCursor & operator=(const GeoVolumeCursor &right) = delete;
+  virtual ~GeoVolumeCursor() = default;
 
   /// Advance
   void next();
@@ -79,29 +76,28 @@ class GeoVolumeCursor final : public GeoNodeAction
   /// Ressucitate (undo terminate)
   void resuscitate();
 
-
-  PVConstLink                           m_parent;
-  PVConstLink                           m_volume;
-  VSConstLink                           m_surface;
-  GeoTrf::Transform3D                           m_transform;
-  GeoTrf::Transform3D                           m_defTransform;
+  PVConstLink                           m_parent{nullptr};
+  PVConstLink                           m_volume{nullptr};
+  VSConstLink                           m_surface{nullptr};
+  GeoTrf::Transform3D                   m_transform{GeoTrf::Transform3D::Identity()};
+  GeoTrf::Transform3D                   m_defTransform{GeoTrf::Transform3D::Identity()};
   
-  unsigned int                          m_majorIndex;
-  unsigned int                          m_minorIndex;
-  unsigned int                          m_minorLimit;
-  const GeoSerialTransformer           *m_serialTransformer;
+  unsigned int                          m_majorIndex{0};
+  unsigned int                          m_minorIndex{0};
+  unsigned int                          m_minorLimit{0};
+  const GeoSerialTransformer*           m_serialTransformer{nullptr};
   
-  const GeoNameTag                     *m_nameTag;
-  const GeoSerialDenominator           *m_serialDenominator;
-  const GeoIdentifierTag               *m_idTag;
-  std::vector<const GeoTransform *>     m_pendingTransformList;
-  unsigned int                          m_serialDenomPosition;
-  const GeoSerialIdentifier            *m_serialIdentifier;
-  unsigned int                          m_serialIdentPosition;
-  unsigned int                          m_volCount;
-  bool                                  m_hasAlignTrans;
-
-  GeoVAlignmentStore                   *m_alignStore;
+  const GeoNameTag*                     m_nameTag{nullptr};
+  const GeoSerialDenominator*           m_serialDenominator{nullptr};
+  const GeoIdentifierTag*               m_idTag{nullptr};
+  std::vector<const GeoTransform *>     m_pendingTransformList{};
+  unsigned int                          m_serialDenomPosition{0};
+  const GeoSerialIdentifier*            m_serialIdentifier{nullptr};
+  unsigned int                          m_serialIdentPosition{0};
+  unsigned int                          m_volCount{0};
+  bool                                  m_hasAlignTrans{false};
+
+  GeoVAlignmentStore*                   m_alignStore{};
 };
 
 #endif

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVolumeTagCatalog.h

@@ -4,21 +4,20 @@
 #include <map>
 #include "GeoModelKernel/GeoVPhysVol.h"
 
-#define volumeTags std::map<std::string, GeoVPhysVol* >
+#define volumeTags std::map<std::string, PVLink >
 #define tagCatalog std::map<std::string, volumeTags >
 
 class GeoVolumeTagCatalog {
 public:
-	void addTaggedVolume(std::string category, std::string tag, GeoVPhysVol* v)
-	{
+	void addTaggedVolume(const std::string& category, const std::string& tag, PVLink v) {
 		theTags[category][tag]=v;	
 	} 
-	GeoVPhysVol* getTaggedVolume(std::string category, std::string tag)
+	GeoVPhysVol* getTaggedVolume(const std::string& category, const std::string& tag) 
 	{
 		return theTags[category][tag];
 	}
 
-	GeoVolumeTagCatalog() {}
+	GeoVolumeTagCatalog() = default;
 	static GeoVolumeTagCatalog* VolumeTagCatalog() {
 		static GeoVolumeTagCatalog* theCatalog=new GeoVolumeTagCatalog;
 		return theCatalog;

GeoModelCore/GeoModelKernel/src/GeoAccessSurfaceAction.cxx

@@ -16,8 +16,6 @@ GeoAccessSurfaceAction::GeoAccessSurfaceAction(unsigned int Index, const GeoVAli
   m_pendingTransformList.reserve(1);
 }
 
-GeoAccessSurfaceAction::~GeoAccessSurfaceAction() = default;
-
 void GeoAccessSurfaceAction::handleTransform (const GeoTransform *xform)
 {
   m_pendingTransformList.push_back (xform);

GeoModelCore/GeoModelKernel/src/GeoClearAbsPosAction.cxx

@@ -4,15 +4,7 @@
 
 #include "GeoModelKernel/GeoClearAbsPosAction.h"
 
-GeoClearAbsPosAction::GeoClearAbsPosAction()
-{
-}
-
-GeoClearAbsPosAction::~GeoClearAbsPosAction()
-{
-}
 
-void GeoClearAbsPosAction::handleFullPhysVol (const GeoFullPhysVol *vol)
-{
-  vol->clearPositionInfo ();
+void GeoClearAbsPosAction::handleFullPhysVol (const GeoFullPhysVol *vol) {
+  vol->clearPositionInfo();
 }

GeoModelCore/GeoModelKernel/src/GeoCountSurfaceAction.cxx

@@ -9,8 +9,6 @@ GeoCountSurfaceAction::GeoCountSurfaceAction ()
   setDepthLimit (1);
 }
 
-GeoCountSurfaceAction::~GeoCountSurfaceAction() = default;
-
 void GeoCountSurfaceAction::handleVSurface (const GeoVSurface *)
 {
   // count the number of all virtual surfaces.    

GeoModelCore/GeoModelKernel/src/GeoCountVolAction.cxx

@@ -5,14 +5,10 @@
 #include "GeoModelKernel/GeoCountVolAction.h"
 
 GeoCountVolAction::GeoCountVolAction ()
- : m_count (0)
 {
   setDepthLimit (1);
 }
 
-GeoCountVolAction::~GeoCountVolAction()
-{
-}
 
 void GeoCountVolAction::handlePhysVol (const GeoPhysVol *)
 {
@@ -20,7 +16,7 @@ void GeoCountVolAction::handlePhysVol (const GeoPhysVol *)
   // Do not count the top volume, this action counts only children!   
   //   
   if (getPath ()->getLength () > 1)
-    m_count++;
+    ++m_count;
 }
 
 void GeoCountVolAction::handleFullPhysVol (const GeoFullPhysVol *)
@@ -29,13 +25,13 @@ void GeoCountVolAction::handleFullPhysVol (const GeoFullPhysVol *)
   // Do not count the top volume, this action counts only children!   
   //   
   if (getPath ()->getLength () > 1)
-    m_count++;
+    ++m_count;
 }
 
 void GeoCountVolAction::handleVSurface (const GeoVSurface *)
 {
   // Virtual Surface is always child of Physical Volume  
-  m_count++;
+  ++m_count;
 }
 
 

GeoModelCore/GeoModelKernel/src/GeoCountVolAndSTAction.cxx

@@ -4,15 +4,10 @@
 
 #include "GeoModelKernel/GeoCountVolAndSTAction.h"
 
-GeoCountVolAndSTAction::GeoCountVolAndSTAction()
-  :m_count (0)
-{
+GeoCountVolAndSTAction::GeoCountVolAndSTAction() {
   setDepthLimit (1);
 }
 
-GeoCountVolAndSTAction::~GeoCountVolAndSTAction()
-{
-}
 
 void GeoCountVolAndSTAction::handlePhysVol(const GeoPhysVol*)
 {
@@ -20,7 +15,7 @@ void GeoCountVolAndSTAction::handlePhysVol(const GeoPhysVol*)
   // Do not count the top volume, this action counts only children!   
   //   
   if (getPath ()->getLength () > 1)
-    m_count++;
+    ++m_count;
 }
 
 void GeoCountVolAndSTAction::handleFullPhysVol(const GeoFullPhysVol*)
@@ -29,12 +24,11 @@ void GeoCountVolAndSTAction::handleFullPhysVol(const GeoFullPhysVol*)
   // Do not count the top volume, this action counts only children!   
   //   
   if (getPath ()->getLength () > 1)
-    m_count++;
+    ++m_count;
 }
 
 void GeoCountVolAndSTAction::handleSerialTransformer(const GeoSerialTransformer *)
 {
-  m_count ++;
 }