Optimize header inclusion

src/LA/system_solvers_large.hpp

@@ -70,6 +70,8 @@ struct KSPStatus {
 class SystemMatrixAssembler {
 
 public:
+    virtual ~SystemMatrixAssembler() = default;
+
     virtual long getRowCount() const = 0;
     virtual long getColCount() const = 0;
 

src/PABLO/LocalTree.hpp

@@ -31,6 +31,7 @@
 #include "tree_constants.hpp"
 #include "Octant.hpp"
 #include "Intersection.hpp"
+#include "morton.hpp"
 
 namespace bitpit {
 

src/RBF/rbf.cpp

@@ -55,14 +55,6 @@ namespace bitpit {
  *
  */
 
-/*!
- * Destructor
- */
-RBFKernel::~RBFKernel()
-{
-    m_fPtr = NULL;
-}
-
 /*!
  * Default constructor. RBFBasisFunction is WENDLANDC2 by default. RBFMode is
  * INTERP, by default. Use setFunction and setMode for changing it.

src/RBF/rbf.hpp

@@ -81,10 +81,11 @@ class RBFKernel{
     int m_nodes;                                    /**<Number of RBF nodes.*/
 
 public:
-    ~RBFKernel();
     RBFKernel();
     RBFKernel(const RBFKernel & other);
 
+    virtual ~RBFKernel() = default;
+
     void                    setFunction(RBFBasisFunction);
     void                    setFunction(double (&funct)(double ));
 

src/containers/piercedStorageIterator.hpp

@@ -30,8 +30,6 @@
 #include <limits>
 #include <type_traits>
 
-#include "piercedKernel.hpp"
-
 #define  __PSI_REFERENCE__ typename PiercedStorageIterator<value_t, id_t, value_no_cv_t>::reference
 #define  __PSI_POINTER__   typename PiercedStorageIterator<value_t, id_t, value_no_cv_t>::pointer
 

src/containers/piercedSync.hpp

@@ -105,6 +105,9 @@ class PiercedSyncSlave {
 
 friend class PiercedSyncMaster;
 
+public:
+    virtual ~PiercedSyncSlave() = default;
+
 protected:
     PiercedSyncSlave();
 

src/levelset/levelSetBoundedObject.hpp

@@ -32,6 +32,8 @@ namespace bitpit {
 class LevelSetBoundedObject {
 
 public:
+    virtual ~LevelSetBoundedObject() = default;
+
     virtual void                                getBoundingBox( std::array<double,3> &, std::array<double,3> & )const =0  ;
 # if BITPIT_ENABLE_MPI
     virtual void                                getGlobalBoundingBox( std::array<double,3> &, std::array<double,3> & )const =0  ;

src/levelset/levelSetCachedObject.hpp

@@ -32,7 +32,6 @@
 
 # include "bitpit_containers.hpp"
 
-# include "levelSetBoundedObject.hpp"
 # include "levelSetKernel.hpp"
 # include "levelSetSignedObject.hpp"
 # include "levelSetStorage.hpp"

src/levelset/levelSetMaskObject.hpp

@@ -32,8 +32,6 @@
 # include "bitpit_operators.hpp"
 # include "bitpit_surfunstructured.hpp"
 
-# include "levelSetCartesianKernel.hpp"
-# include "levelSetOctreeKernel.hpp"
 # include "levelSetSegmentationObject.hpp"
 
 namespace bitpit{

src/levelset/levelSetObject.hpp

@@ -52,6 +52,8 @@ class LevelSetKernel;
 class LevelSetObjectInterface {
 
     public:
+    virtual ~LevelSetObjectInterface() = default;
+
     virtual LevelSetKernel *                    getKernel() = 0;
     virtual const LevelSetKernel *              getKernel() const = 0;
     virtual void                                setKernel(LevelSetKernel *) = 0;

src/levelset/levelSetSegmentationObject.hpp

@@ -37,6 +37,7 @@
 #include "bitpit_voloctree.hpp"
 
 #include "levelSetCommon.hpp"
+#include "levelSetBoundedObject.hpp"
 #include "levelSetCachedObject.hpp"
 #include "levelSetCartesianKernel.hpp"
 #include "levelSetOctreeKernel.hpp"

src/patchkernel/volume_mapper.hpp

@@ -32,7 +32,7 @@
 #include <vector>
 #include <unordered_map>
 
-#include "bitpit_patchkernel.hpp"
+#include "volume_kernel.hpp"
 
 namespace bitpit {