Fix some warnings (#1239)

This PR fixes the source of several compilation warnings.

Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>

4 files changed, 22 insertions, 17 deletions

diff --git a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/AvgGalileanAlgorithm.cpp b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/AvgGalileanAlgorithm.cpp
index e5fbc8261..de02c0970 100644
--- a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/AvgGalileanAlgorithm.cpp
+++ b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/AvgGalileanAlgorithm.cpp
@@ -225,7 +225,7 @@ void AvgGalileanAlgorithm::InitializeSpectralCoefficients(
 
         });
     }
-};
+}
 
 /* Advance the E and B field in spectral space (stored in `f`)
  * over one time step */
@@ -368,19 +368,19 @@ AvgGalileanAlgorithm::pushSpectralFields(SpectralFieldData& f) const{
                           + A1*I*(kx*Jy     - ky*Jx)*inv_ep0;
                         });
     }
-};
+}
 
 void
-AvgGalileanAlgorithm::CurrentCorrection (SpectralFieldData& field_data,
-                                         std::array<std::unique_ptr<amrex::MultiFab>,3>& current,
-                                         const std::unique_ptr<amrex::MultiFab>& rho)
+AvgGalileanAlgorithm::CurrentCorrection (SpectralFieldData& /*field_data*/,
+                                         std::array<std::unique_ptr<amrex::MultiFab>,3>& /*current*/,
+                                         const std::unique_ptr<amrex::MultiFab>& /*rho*/)
 {
     amrex::Abort("Current correction not implemented for averaged Galilean PSATD");
 }
 
 void
-AvgGalileanAlgorithm::VayDeposition (SpectralFieldData& field_data,
-                                     std::array<std::unique_ptr<amrex::MultiFab>,3>& current)
+AvgGalileanAlgorithm::VayDeposition (SpectralFieldData& /*field_data*/,
+                                     std::array<std::unique_ptr<amrex::MultiFab>,3>& /*current*/)
 {
     amrex::Abort("Vay deposition not implemented for averaged Galilean PSATD");
 }
diff --git a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/PMLPsatdAlgorithm.cpp b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/PMLPsatdAlgorithm.cpp
index 4bc147cd8..e1be557fe 100644
--- a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/PMLPsatdAlgorithm.cpp
+++ b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/PMLPsatdAlgorithm.cpp
@@ -103,7 +103,7 @@ PMLPsatdAlgorithm::pushSpectralFields(SpectralFieldData& f) const{
             fields(i,j,k,Idx::Bzy) = C*fields(i,j,k,Idx::Bzy) + S_ck*I*ky*Ex_old;
         });
     }
-};
+}
 
 void PMLPsatdAlgorithm::InitializeSpectralCoefficients (
     const SpectralKSpace& spectral_kspace,
@@ -153,19 +153,19 @@ void PMLPsatdAlgorithm::InitializeSpectralCoefficients (
             }
         });
     }
-};
+}
 
 void
-PMLPsatdAlgorithm::CurrentCorrection (SpectralFieldData& field_data,
-                                      std::array<std::unique_ptr<amrex::MultiFab>,3>& current,
-                                      const std::unique_ptr<amrex::MultiFab>& rho)
+PMLPsatdAlgorithm::CurrentCorrection (SpectralFieldData& /*field_data*/,
+                                      std::array<std::unique_ptr<amrex::MultiFab>,3>& /*current*/,
+                                      const std::unique_ptr<amrex::MultiFab>& /*rho*/)
 {
     amrex::Abort("Current correction not implemented for PML PSATD");
 }
 
 void
-PMLPsatdAlgorithm::VayDeposition (SpectralFieldData& field_data,
-                                  std::array<std::unique_ptr<amrex::MultiFab>,3>& current)
+PMLPsatdAlgorithm::VayDeposition (SpectralFieldData& /*field_data*/,
+                                  std::array<std::unique_ptr<amrex::MultiFab>,3>& /*current*/)
 {
     amrex::Abort("Vay deposition not implemented for PML PSATD");
 }
diff --git a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.H b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.H
index 73ad773f3..3e909145d 100644
--- a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.H
+++ b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.H
@@ -10,6 +10,7 @@
 #include "FieldSolver/SpectralSolver/SpectralKSpace.H"
 #include "FieldSolver/SpectralSolver/SpectralFieldData.H"
 
+#include <AMReX.H>
 
 #if WARPX_USE_PSATD
 
@@ -29,7 +30,7 @@ class SpectralBaseAlgorithm
         // The destructor should also be a virtual function, so that
         // a pointer to subclass of `SpectraBaseAlgorithm` actually
         // calls the subclass's destructor.
-        virtual ~SpectralBaseAlgorithm() {};
+        virtual ~SpectralBaseAlgorithm() {}
 
         /**
          * \brief Virtual function for current correction in Fourier space
@@ -84,7 +85,11 @@ class SpectralBaseAlgorithm
 #else
             modified_kz_vec(spectral_kspace.getModifiedKComponent(dm,1,norder_z,nodal))
 #endif
-          {};
+          {
+#if (AMREX_SPACEDIM!=3)
+            amrex::ignore_unused(norder_y);
+#endif
+          }
 
         // Modified finite-order vectors
         KVectorComponent modified_kx_vec;
diff --git a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.cpp b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.cpp
index 73a0bb9af..3354384f0 100644
--- a/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.cpp
+++ b/Source/FieldSolver/SpectralSolver/SpectralAlgorithms/SpectralBaseAlgorithm.cpp
@@ -66,4 +66,4 @@ SpectralBaseAlgorithm::ComputeSpectralDivE (
 
     // Backward Fourier transform
     field_data.BackwardTransform( divE, Idx::divE, 0 );
-};
+}