From d9e45cb58c65349eb8de973c00383447402ea95e Mon Sep 17 00:00:00 2001
From: Hrvoje Jasak <h.jasak@wikki.co.uk>
Date: Thu, 18 Oct 2018 19:18:19 +0100
Subject: [PATCH] Clean-up of default copy constructors for polyPatches - copy
 constructor needed

---
 .../passiveProcessorPolyPatch.C               | 38 ++++++++++++-------
 .../passiveProcessorPolyPatch.H               | 17 +++++----
 2 files changed, 35 insertions(+), 20 deletions(-)

diff --git a/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.C b/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.C
index 40ac98547..41d4c6be7 100644
--- a/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.C
+++ b/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.C
@@ -96,19 +96,6 @@ Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
 {}
 
 
-Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
-(
-    const passiveProcessorPolyPatch& pp,
-    const polyBoundaryMesh& bm
-)
-:
-    polyPatch(pp, bm),
-    myProcNo_(pp.myProcNo_),
-    neighbProcNo_(pp.neighbProcNo_),
-    globalFaceIndex_(pp.size(), -1)
-{}
-
-
 Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
 (
     const passiveProcessorPolyPatch& pp,
@@ -125,6 +112,31 @@ Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
 {}
 
 
+Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
+(
+    const passiveProcessorPolyPatch& pp
+)
+:
+    polyPatch(pp),
+    myProcNo_(pp.myProcNo_),
+    neighbProcNo_(pp.neighbProcNo_),
+    globalFaceIndex_(pp.size(), -1)
+{}
+
+
+Foam::passiveProcessorPolyPatch::passiveProcessorPolyPatch
+(
+    const passiveProcessorPolyPatch& pp,
+    const polyBoundaryMesh& bm
+)
+:
+    polyPatch(pp, bm),
+    myProcNo_(pp.myProcNo_),
+    neighbProcNo_(pp.neighbProcNo_),
+    globalFaceIndex_(pp.size(), -1)
+{}
+
+
 // * * * * * * * * * * * * * * * * Destructor  * * * * * * * * * * * * * * * //
 
 Foam::passiveProcessorPolyPatch::~passiveProcessorPolyPatch()
diff --git a/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.H b/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.H
index e15bb0e98..390a42cb1 100644
--- a/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.H
+++ b/src/decompositionMethods/decomposeReconstruct/passivePatches/passiveProcessorPolyPatch/passiveProcessorPolyPatch.H
@@ -99,13 +99,6 @@ public:
             const polyBoundaryMesh&
         );
 
-        //- Construct as copy, resetting the boundary mesh
-        passiveProcessorPolyPatch
-        (
-            const passiveProcessorPolyPatch&,
-            const polyBoundaryMesh&
-        );
-
         //- Construct as given the original patch and resetting the
         //  face list and boundary mesh information
         passiveProcessorPolyPatch
@@ -117,6 +110,16 @@ public:
             const label newStart
         );
 
+        //- Construct as copy
+        passiveProcessorPolyPatch(const passiveProcessorPolyPatch&);
+
+        //- Construct as copy, resetting the boundary mesh
+        passiveProcessorPolyPatch
+        (
+            const passiveProcessorPolyPatch&,
+            const polyBoundaryMesh&
+        );
+
         //- Construct and return a clone, resetting the boundary mesh
         virtual autoPtr<polyPatch> clone(const polyBoundaryMesh& bm) const
         {