Apply suggestions from code review

ax3l · Mar 28, 2024 · db8a619 · db8a619
1 parent a1686f3
commit db8a619
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Src/Base/AMReX_MultiFab.H b/Src/Base/AMReX_MultiFab.H
@@ -492,7 +492,7 @@ public:
      * This uses the same arena and factory in the copy and copies
      * all data.
      */
-    MultiFab deepCopy ();
+    [[nodiscard]] MultiFab deepCopy () const;
 
     /**
     * \brief Copy from src to dst including nghost ghost cells.

diff --git a/Src/Base/AMReX_MultiFab.cpp b/Src/Base/AMReX_MultiFab.cpp
@@ -168,9 +168,9 @@ MultiFab::Add (MultiFab& dst, const MultiFab& src,
 }
 
 MultiFab
-MultiFab::deepCopy ()
+MultiFab::deepCopy () const
 {
-    auto mf = MultiFab(
+    MultiFab mf(
         this->boxArray(),
         this->DistributionMap(),
         this->nComp(),