Fix warnings

WeiqunZhang · Jan 4, 2024 · a33253c · a33253c
1 parent b4193a4
commit a33253c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/Src/Extern/openPMD-api/AMReX_PlotFileUtilOPENPMD.H b/Src/Extern/openPMD-api/AMReX_PlotFileUtilOPENPMD.H
@@ -19,10 +19,8 @@
 #include <string>
 #include <memory>
 
-namespace amrex
+namespace amrex::openpmd_api
 {
- namespace openpmd_api
- {
  namespace helper
  {
  inline void
@@ -426,8 +424,7 @@ namespace amrex
  }; // class AMReX_openPMDHandler
 
 
- }// name space openpmd_api
-} // namespace amrex
+} // namespace amrex::openpmd_api
 
 
 #include <AMReX_PlotFileUtilOPENPMD_PTLImpl.H>

diff --git a/Src/Extern/openPMD-api/AMReX_PlotFileUtilOPENPMD_PTLImpl.H b/Src/Extern/openPMD-api/AMReX_PlotFileUtilOPENPMD_PTLImpl.H
@@ -2,10 +2,8 @@
 #define AMREX_PLOTFILE_OPENPMD_PTLS_API_H
 
 //extern std::unique_ptr< amrex::openpmd_api::AMReX_openPMDHandler > amrex::openpmd_api::m_OpenPMDHandler;
-namespace amrex
+namespace amrex::openpmd_api
 {
- namespace openpmd_api
- {
  extern std::unique_ptr< amrex::openpmd_api::AMReX_openPMDHandler > m_OpenPMDHandler;
  ////////////////////////////////////////////////////////////////////////
  //
@@ -426,7 +424,7 @@ namespace amrex
  //amrex::AllPrint()<<" i = "<<i<<" "<<aos[i]<<"\n";
  curr.get()[i] = aos[i].pos(currDim);
  }
- std::string const positionComponent = positionComponents[currDim];
+ std::string const& positionComponent = positionComponents[currDim];
  currSpecies["position"][positionComponent].storeChunk(curr, {offset + GetGrandOffset()}, {numParticleOnTile64});
  }
  }
@@ -658,7 +656,6 @@ namespace amrex
  }
  }
 
- } // namespace openpmd_api
-} // namespace amrex
+} // namespace amrex::openpmd_api
 
 #endif // AMREX_PLOTFILE_OPENPMD_API_H