Minor constructor clean up

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 *) override;
@@ -29,8 +29,6 @@ class GeoCountVolAction : public GeoNodeAction
   unsigned int getCount () const;
 
  private:
-  GeoCountVolAction(const GeoCountVolAction &right);
-  GeoCountVolAction & operator=(const GeoCountVolAction &right);
 
   unsigned int m_count{};
 };

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,9 +27,6 @@ class GeoCountVolAndSTAction : public GeoNodeAction
   unsigned int getCount() const;
 
  private:
-  GeoCountVolAndSTAction(const GeoCountVolAndSTAction &right);
-  const GeoCountVolAndSTAction & operator=(const GeoCountVolAndSTAction &right);
-
   unsigned int m_count{};
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoShapeAction.h

@@ -138,8 +138,8 @@ 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.
@@ -147,7 +147,7 @@ class GeoShapeAction
   
   //	Termination flag; causes an abortion of action execution.
   bool m_terminate{};
-  GeoShapePath m_path;
+  GeoShapePath m_path{};
 
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoSurfaceCursor.h

@@ -61,7 +61,7 @@ class GeoSurfaceCursor final : public GeoNodeAction
   unsigned int                          m_volCount{};
   unsigned int                          m_surfCount{};
    
-  std::vector<const GeoTransform *>     m_pendingTransformList;
+  std::vector<const GeoTransform *>     m_pendingTransformList{};
   
   bool                                  m_hasAlignTrans{};  
   GeoVAlignmentStore                   *m_alignStore{};

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

@@ -31,6 +31,9 @@ class GeoVDetectorManager
   GeoVDetectorManager();
   virtual ~GeoVDetectorManager();
 
+  GeoVDetectorManager(const GeoVDetectorManager &right) = delete;
+  GeoVDetectorManager & operator=(const GeoVDetectorManager &right) = delete;
+
   //	Returns the number of Tree Tops (Top Level Physicsl
   //	Volumes)
   virtual unsigned int getNumTreeTops () const = 0;
@@ -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,8 +50,6 @@ 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.
@@ -60,7 +58,7 @@ class GeoVolumeAction
   //	Termination flag; causes an abortion of action execution.
   bool m_terminate{};
 
-  GeoTraversalState m_traversalState;
+  GeoTraversalState m_traversalState{};
   
 };
 

GeoModelCore/GeoModelKernel/GeoModelKernel/GeoVolumeCursor.h

@@ -19,10 +19,7 @@ class GeoVolumeCursor final : public GeoNodeAction
   friend GeoVolumeCursorTest;
   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();

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

@@ -8,15 +8,12 @@ GeoCountVolAction::GeoCountVolAction (){
   setDepthLimit (1);
 }
 
-GeoCountVolAction::~GeoCountVolAction(){
-}
-
 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 *){
@@ -24,12 +21,12 @@ 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

@@ -8,15 +8,12 @@ GeoCountVolAndSTAction::GeoCountVolAndSTAction(){
   setDepthLimit (1);
 }
 
-GeoCountVolAndSTAction::~GeoCountVolAndSTAction(){
-}
-
 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*){
@@ -24,11 +21,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 ++;
+  ++m_count;
 }