From 47062b614cc2a7b3db3b2d454380c655e899cfe9 Mon Sep 17 00:00:00 2001
From: Mark Olesen <Mark.Olesen@esi-group.com>
Date: Thu, 30 Nov 2023 11:18:28 +0000
Subject: [PATCH] GIT: remove supplementary IOobjectOption handling for faMesh

- was previously used to suppress loading of faSchemes/faSolution,
  but now superfluous.
---
 .../distributed/faMeshDistributorNew.C        |  6 +-----
 src/finiteArea/faMesh/faMesh.C                | 13 +-----------
 src/finiteArea/faMesh/faMesh.H                | 11 ----------
 .../faMesh/faMeshTools/faMeshTools.C          | 21 ++++---------------
 4 files changed, 6 insertions(+), 45 deletions(-)

diff --git a/src/finiteArea/distributed/faMeshDistributorNew.C b/src/finiteArea/distributed/faMeshDistributorNew.C
index cc6efdd4238..d2adcca84d8 100644
--- a/src/finiteArea/distributed/faMeshDistributorNew.C
+++ b/src/finiteArea/distributed/faMeshDistributorNew.C
@@ -93,13 +93,9 @@ Foam::faMeshDistributor::distribute
             ListOps::findIndices(oldToNewConstruct, labelRange::ge0())
         );
 
-        // Set up to read-if-present
-        IOobject io(tgtPolyMesh);
-        io.readOpt(IOobject::READ_IF_PRESENT);
-
         newMeshPtr.reset
         (
-            new faMesh(tgtPolyMesh, std::move(newFaceLabels), io)
+            new faMesh(tgtPolyMesh, std::move(newFaceLabels))
         );
     }
 
diff --git a/src/finiteArea/faMesh/faMesh.C b/src/finiteArea/faMesh/faMesh.C
index b9e13ffdf6c..ee70a14553c 100644
--- a/src/finiteArea/faMesh/faMesh.C
+++ b/src/finiteArea/faMesh/faMesh.C
@@ -325,7 +325,7 @@ bool Foam::faMesh::init(const bool doInit)
 
 Foam::faMesh::faMesh(const polyMesh& pMesh, const Foam::zero)
 :
-    faMesh(pMesh, labelList(), static_cast<const IOobject&>(pMesh))
+    faMesh(pMesh, labelList())
 {}
 
 
@@ -449,17 +449,6 @@ Foam::faMesh::faMesh
     const polyMesh& pMesh,
     labelList&& faceLabels
 )
-:
-    faMesh(pMesh, std::move(faceLabels), static_cast<const IOobject&>(pMesh))
-{}
-
-
-Foam::faMesh::faMesh
-(
-    const polyMesh& pMesh,
-    labelList&& faceLabels,
-    const IOobject& io
-)
 :
     MeshObject<polyMesh, Foam::UpdateableMeshObject, faMesh>(pMesh),
     edgeInterpolation(*this),
diff --git a/src/finiteArea/faMesh/faMesh.H b/src/finiteArea/faMesh/faMesh.H
index ea66093ff7d..87a487f8e6c 100644
--- a/src/finiteArea/faMesh/faMesh.H
+++ b/src/finiteArea/faMesh/faMesh.H
@@ -552,17 +552,6 @@ public:
         //  Boundary is added using addFaPatches() member function.
         faMesh(const polyMesh& pMesh, labelList&& faceLabels);
 
-        //- Construct from components (face labels) without boundary,
-        //- using alternative IOobject properties
-        //- (primarily the readOption).
-        //  Boundary is added using addFaPatches() member function.
-        faMesh
-        (
-            const polyMesh& pMesh,
-            labelList&& faceLabels,
-            const IOobject& io
-        );
-
         //- Construct from single polyPatch
         explicit faMesh(const polyPatch& pp, const bool doInit = true);
 
diff --git a/src/finiteArea/faMesh/faMeshTools/faMeshTools.C b/src/finiteArea/faMesh/faMeshTools/faMeshTools.C
index d9a346a52d1..997dc409abb 100644
--- a/src/finiteArea/faMesh/faMeshTools/faMeshTools.C
+++ b/src/finiteArea/faMesh/faMeshTools/faMeshTools.C
@@ -136,12 +136,9 @@ Foam::faMeshTools::newMesh
     // Set up to read-if-present. Note: does not search for mesh so set
     // instance explicitly
 
-    IOobject meshIO(io);
-    meshIO.instance() = facesInstance;
-    meshIO.readOpt(IOobject::READ_IF_PRESENT);
-
     // For mesh components (faceLabels, ...)
-    IOobject cmptIO(meshIO, "faceLabels", meshSubDir);
+    IOobject cmptIO(io, "faceLabels", meshSubDir);
+    cmptIO.instance() = facesInstance;
     cmptIO.readOpt(IOobject::MUST_READ);
     cmptIO.writeOpt(IOobject::NO_WRITE);
     cmptIO.registerObject(false);
@@ -154,7 +151,6 @@ Foam::faMeshTools::newMesh
     if (masterOnlyReading && !UPstream::master())
     {
         haveMesh = false;
-        meshIO.readOpt(IOobject::NO_READ);
     }
 
     if (!haveMesh)
@@ -182,8 +178,7 @@ Foam::faMeshTools::newMesh
     auto meshPtr = autoPtr<faMesh>::New
     (
         pMesh,
-        std::move(faceLabels),
-        meshIO
+        std::move(faceLabels)
     );
     auto& mesh = *meshPtr;
 
@@ -354,15 +349,7 @@ Foam::faMeshTools::loadOrCreateMeshImpl
         const int oldCache = fileOperation::cacheLevel(0);
 
         // Create dummy mesh - on procs that don't already have a mesh
-        meshPtr.reset
-        (
-            new faMesh
-            (
-                pMesh,
-                labelList(),
-                IOobject(io, IOobject::NO_READ, IOobject::AUTO_WRITE)
-            )
-        );
+        meshPtr.reset(new faMesh(pMesh, labelList()));
         faMesh& mesh = *meshPtr;
 
         // Add patches
-- 
GitLab