]> SALOME platform Git repositories - tools/medcoupling.git/commitdiff
Salome HOME
Remove useless usage of dynamic allocation into BoundingBox class
authorAnthony Geay <anthony.geay@edf.fr>
Fri, 19 Aug 2022 12:53:09 +0000 (14:53 +0200)
committerAnthony Geay <anthony.geay@edf.fr>
Fri, 19 Aug 2022 12:53:09 +0000 (14:53 +0200)
src/INTERP_KERNEL/BoundingBox.cxx
src/INTERP_KERNEL/BoundingBox.hxx

index ab192e4f6ea7d6a7361195accde04f54f3299896..db5fd091850b2c5a5d0f48cdd434197939a7d1bf 100644 (file)
@@ -36,10 +36,7 @@ namespace INTERP_KERNEL
    *
    */
   BoundingBox::BoundingBox(const double** pts, const unsigned numPts)
-    :_coords(new double[6])
   {
-    assert(numPts > 0);     
-
     // initialize with first two points
     const double *pt0(pts[0]);
 
@@ -63,11 +60,8 @@ namespace INTERP_KERNEL
    * @param  box1  the first box
    * @param  box2  the second box
    */
-  BoundingBox::BoundingBox(const BoundingBox& box1, const BoundingBox& box2) 
-    : _coords(new double[6])
+  BoundingBox::BoundingBox(const BoundingBox& box1, const BoundingBox& box2)
   {
-    assert(_coords != 0);
-
     for(BoxCoord c = XMIN ; c <= ZMIN ; c = BoxCoord(c + 1))
       {
         _coords[c] = std::min(box1._coords[c], box2._coords[c]);
@@ -77,15 +71,6 @@ namespace INTERP_KERNEL
     assert(isValid());
   }
 
-  /**
-   * Destructor
-   *
-   */
-  BoundingBox::~BoundingBox()
-  {
-    delete [] _coords;
-  }
-
   /**
    * Determines if the intersection with a given box is empty
    * 
index d453e8a62a4ebdb58b999d9b18556f48d9045ce8..52641ccf347f65e58cfb7fc22c4619dfdeab6573 100644 (file)
@@ -41,7 +41,7 @@ namespace INTERP_KERNEL
 
     BoundingBox(const BoundingBox& box1, const BoundingBox& box2);
 
-    ~BoundingBox();
+    ~BoundingBox() { }
 
     bool isDisjointWith(const BoundingBox& box) const;
     
@@ -67,7 +67,7 @@ namespace INTERP_KERNEL
     
     /// Vector containing the coordinates of the box
     /// interlaced in the order XMIN, YMIN, ZMIN, XMAX, YMAX, ZMAX
-    double* _coords;
+    double _coords[6];
 
   };