MFIter: Make Finalize Public (AMReX-Codes#2985)

Follow-up to AMReX-Codes#2983
Thierry992 · Oct 12, 2022 · ed1ecd6 · ed1ecd6
1 parent 5acfe07
commit ed1ecd6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/Src/Base/AMReX_MFIter.H b/Src/Base/AMReX_MFIter.H
@@ -164,6 +164,8 @@ public:
 
     static int allowMultipleMFIters (int allow);
 
+    void Finalize ();
+
 protected:
 
     std::unique_ptr<FabArrayBase> m_fa;  //!< This must be the first member!
@@ -202,7 +204,6 @@ protected:
     static AMREX_EXPORT int allow_multiple_mfiters;
 
     void Initialize ();
-    void Finalize ();
 };
 
 //! Is it safe to have these two MultiFabs in the same MFiter?

diff --git a/Src/Base/AMReX_MFIter.cpp b/Src/Base/AMReX_MFIter.cpp
@@ -215,9 +215,13 @@ MFIter::~MFIter ()
 void
 MFIter::Finalize ()
 {
+    // avoid double finalize
     if (finalized) return;
     finalized = true;
 
+    // mark as invalid
+    currentIndex = endIndex;
+
 #ifdef AMREX_USE_OMP
 #pragma omp master
 #endif