[MGTL] Rm default args.; Use enum SearchAllNodes.

MeshGeoToolsLib/AppendLinesAlongPolyline.cpp

@@ -53,7 +53,9 @@ std::unique_ptr<MeshLib::Mesh> appendLinesAlongPolylines(
         const GeoLib::Polyline* ply = (*ply_vec.getVector())[k];
 
         // search nodes on the polyline
-        MeshGeoToolsLib::MeshNodesAlongPolyline mshNodesAlongPoly(mesh, *ply, mesh.getMinEdgeLength()*0.5);
+        MeshGeoToolsLib::MeshNodesAlongPolyline mshNodesAlongPoly(
+            mesh, *ply, mesh.getMinEdgeLength() * 0.5,
+            MeshGeoToolsLib::SearchAllNodes::Yes);
         auto &vec_nodes_on_ply = mshNodesAlongPoly.getNodeIDs();
         if (vec_nodes_on_ply.empty()) {
             std::string ply_name;

MeshGeoToolsLib/MeshNodeSearcher.cpp

@@ -109,7 +109,7 @@ MeshNodesOnPoint& MeshNodeSearcher::getMeshNodesOnPoint(
                              _mesh_grid,
                              pnt,
                              _search_length_algorithm.getSearchLength(),
-                             _search_all_nodes == SearchAllNodes::Yes));
+                             _search_all_nodes));
     return *_mesh_nodes_on_points.back();
 }
 
@@ -127,7 +127,7 @@ MeshNodesAlongPolyline& MeshNodeSearcher::getMeshNodesAlongPolyline(
     // compute nodes (and supporting points) along polyline
     _mesh_nodes_along_polylines.push_back(new MeshNodesAlongPolyline(
         _mesh, ply, _search_length_algorithm.getSearchLength(),
-        _search_all_nodes == SearchAllNodes::Yes));
+        _search_all_nodes));
     return *_mesh_nodes_along_polylines.back();
 }
 
@@ -147,7 +147,7 @@ MeshNodesAlongSurface& MeshNodeSearcher::getMeshNodesAlongSurface(
         new MeshNodesAlongSurface(_mesh,
                                   sfc,
                                   _search_length_algorithm.getSearchLength(),
-                                  _search_all_nodes == SearchAllNodes::Yes));
+                                  _search_all_nodes));
     return *_mesh_nodes_along_surfaces.back();
 }
 

MeshGeoToolsLib/MeshNodesAlongPolyline.cpp

@@ -22,15 +22,16 @@
 
 namespace MeshGeoToolsLib
 {
-MeshNodesAlongPolyline::MeshNodesAlongPolyline(
-        MeshLib::Mesh const& mesh,
-        GeoLib::Polyline const& ply,
-        double epsilon_radius,
-        bool search_all_nodes) :
-    _mesh(mesh), _ply(ply)
+MeshNodesAlongPolyline::MeshNodesAlongPolyline(MeshLib::Mesh const& mesh,
+                                               GeoLib::Polyline const& ply,
+                                               double epsilon_radius,
+                                               SearchAllNodes search_all_nodes)
+    : _mesh(mesh), _ply(ply)
 {
     assert(epsilon_radius > 0);
-    const std::size_t n_nodes (search_all_nodes ? _mesh.getNumberOfNodes() : _mesh.getNumberOfBaseNodes());
+    const std::size_t n_nodes(search_all_nodes == SearchAllNodes::Yes
+                                  ? _mesh.getNumberOfNodes()
+                                  : _mesh.getNumberOfBaseNodes());
     auto &mesh_nodes = _mesh.getNodes();
     // loop over all nodes
     for (std::size_t i = 0; i < n_nodes; i++) {

MeshGeoToolsLib/MeshNodesAlongPolyline.h

@@ -14,6 +14,8 @@
 
 #include <vector>
 
+#include "MeshGeoToolsLib/SearchAllNodes.h"
+
 namespace GeoLib
 {
 class Polyline;
@@ -45,9 +47,10 @@ public:
      * @param search_all_nodes switch between searching all mesh nodes and
      * searching the base nodes.
      */
-    MeshNodesAlongPolyline(MeshLib::Mesh const& mesh,
-                           GeoLib::Polyline const& ply, double epsilon_radius,
-                           bool search_all_nodes = true);
+    MeshNodesAlongPolyline(
+        MeshLib::Mesh const& mesh, GeoLib::Polyline const& ply,
+        double epsilon_radius,
+        SearchAllNodes search_all_nodes);
 
     /// return the mesh object
     MeshLib::Mesh const& getMesh() const;

MeshGeoToolsLib/MeshNodesAlongSurface.cpp

@@ -22,16 +22,16 @@
 
 namespace MeshGeoToolsLib
 {
-
-MeshNodesAlongSurface::MeshNodesAlongSurface(
-        MeshLib::Mesh const& mesh,
-        GeoLib::Surface const& sfc,
-        double epsilon_radius,
-        bool search_all_nodes) :
-    _mesh(mesh), _sfc(sfc)
+MeshNodesAlongSurface::MeshNodesAlongSurface(MeshLib::Mesh const& mesh,
+                                             GeoLib::Surface const& sfc,
+                                             double epsilon_radius,
+                                             SearchAllNodes search_all_nodes)
+    : _mesh(mesh), _sfc(sfc)
 {
     auto& mesh_nodes = _mesh.getNodes();
-    const std::size_t n_nodes (search_all_nodes ? _mesh.getNumberOfNodes() : _mesh.getNumberOfBaseNodes());
+    const std::size_t n_nodes(search_all_nodes == SearchAllNodes::Yes
+                                  ? _mesh.getNumberOfNodes()
+                                  : _mesh.getNumberOfBaseNodes());
     // loop over all nodes
     for (std::size_t i = 0; i < n_nodes; i++) {
         auto* node = mesh_nodes[i];

MeshGeoToolsLib/MeshNodesAlongSurface.h

@@ -14,6 +14,8 @@
 
 #include <vector>
 
+#include "MeshGeoToolsLib/SearchAllNodes.h"
+
 namespace GeoLib
 {
 class Surface;
@@ -44,7 +46,7 @@ public:
      * searching the base nodes.
      */
     MeshNodesAlongSurface(MeshLib::Mesh const& mesh, GeoLib::Surface const& sfc,
-                          double epsilon, bool search_all_nodes = true);
+                          double epsilon, SearchAllNodes search_all_nodes);
 
     /// return the mesh object
     MeshLib::Mesh const& getMesh() const;

MeshGeoToolsLib/MeshNodesOnPoint.cpp

@@ -12,13 +12,14 @@
 
 namespace MeshGeoToolsLib
 {
-
-MeshNodesOnPoint::MeshNodesOnPoint(MeshLib::Mesh const& mesh, GeoLib::Grid<MeshLib::Node> const &mesh_grid,
-        GeoLib::Point const& pnt, double epsilon_radius, bool search_all_nodes)
-: _mesh(mesh), _pnt(pnt)
+MeshNodesOnPoint::MeshNodesOnPoint(
+    MeshLib::Mesh const& mesh, GeoLib::Grid<MeshLib::Node> const& mesh_grid,
+    GeoLib::Point const& pnt, double epsilon_radius,
+    SearchAllNodes search_all_nodes)
+    : _mesh(mesh), _pnt(pnt)
 {
     std::vector<std::size_t> vec_ids(mesh_grid.getPointsInEpsilonEnvironment(pnt, epsilon_radius));
-    if (search_all_nodes)
+    if (search_all_nodes == SearchAllNodes::Yes)
         _msh_node_ids = vec_ids;
     else {
         for (auto id : vec_ids)

MeshGeoToolsLib/MeshNodesOnPoint.h

@@ -13,6 +13,8 @@
 #include "GeoLib/Point.h"
 #include "GeoLib/Grid.h"
 
+#include "MeshGeoToolsLib/SearchAllNodes.h"
+
 #include "MeshLib/Node.h"
 
 namespace MeshLib
@@ -35,10 +37,12 @@ public:
      * @param mesh_grid Grid object constructed with mesh nodes
      * @param pnt a point
      * @param epsilon_radius Search radius
-     * @param search_all_nodes whether this shearches all nodes or only base nodes
+     * @param search_all_nodes whether this searches all nodes or only base nodes
      */
-    MeshNodesOnPoint(MeshLib::Mesh const& mesh, GeoLib::Grid<MeshLib::Node> const &mesh_grid,
-            GeoLib::Point const& pnt, double epsilon_radius, bool search_all_nodes = true);
+    MeshNodesOnPoint(MeshLib::Mesh const& mesh,
+                     GeoLib::Grid<MeshLib::Node> const& mesh_grid,
+                     GeoLib::Point const& pnt, double epsilon_radius,
+                     SearchAllNodes search_all_nodes);
 
     /// return the mesh object
     MeshLib::Mesh const& getMesh() const { return _mesh; }