From aa7cce912c324ae0c10c7e40b11bab37ad31ee82 Mon Sep 17 00:00:00 2001
From: Mark Olesen <Mark.Olesen@esi-group.com>
Date: Thu, 1 Nov 2018 14:23:21 +0000
Subject: [PATCH] STYLE: eliminate unnecessary DIY version of cellPoints(label)

---
 .../surface/cutting/cuttingSurfaceCuts.C      | 21 +------------------
 .../surface/distanceSurface/distanceSurface.C |  2 --
 2 files changed, 1 insertion(+), 22 deletions(-)

diff --git a/src/sampling/surface/cutting/cuttingSurfaceCuts.C b/src/sampling/surface/cutting/cuttingSurfaceCuts.C
index 54d65bb880..11558561c1 100644
--- a/src/sampling/surface/cutting/cuttingSurfaceCuts.C
+++ b/src/sampling/surface/cutting/cuttingSurfaceCuts.C
@@ -91,30 +91,11 @@ void Foam::cuttingSurface::calcCellCuts
 
 
     boundBox cellBb;
-    labelHashSet cpts;
 
     for (const label celli : cellCuts)
     {
         cellBb.clear();
-
-        if (fvm.hasCellPoints())
-        {
-            cellBb.add(pts, fvm.cellPoints(celli));
-        }
-        else
-        {
-            // DIY version of cellPoints
-            cpts.clear();
-
-            const cell& cFaces = cells[celli];
-
-            for (const label facei : cFaces)
-            {
-                cpts.insert(faces[facei]);
-            }
-
-            cellBb.add(pts, cpts);
-        }
+        cellBb.add(pts, fvm.cellPoints(celli));
 
         if (!cellBb.contains(nearest[celli].hitPoint()))
         {
diff --git a/src/sampling/surface/distanceSurface/distanceSurface.C b/src/sampling/surface/distanceSurface/distanceSurface.C
index 7b5f5474c5..aa065238b2 100644
--- a/src/sampling/surface/distanceSurface/distanceSurface.C
+++ b/src/sampling/surface/distanceSurface/distanceSurface.C
@@ -202,8 +202,6 @@ void Foam::distanceSurface::createGeometry()
                   : Foam::sign(diff & norms[i]) * Foam::mag(diff)
                 );
 
-                // Since cellPoints() is used in isoSurfaceCell too,
-                // no additional overhead caused here
                 if (filterCells)
                 {
                     cellBb.clear();