Clang tidy CI test: add more readability checks to clang tidy CI test (#4146)

* add more readability checks to clang-tidy CI test

* reformat check list

* starting to address issues found with clang-tidy£

* addressing issues

* remove check

* address issues

* address all issues

* address issue found with github code scanning

1 files changed, 8 insertions, 8 deletions

diff --git a/Source/FieldSolver/SpectralSolver/SpectralHankelTransform/HankelTransform.H b/Source/FieldSolver/SpectralSolver/SpectralHankelTransform/HankelTransform.H
index a911a0627..e40ccc045 100644
--- a/Source/FieldSolver/SpectralSolver/SpectralHankelTransform/HankelTransform.H
+++ b/Source/FieldSolver/SpectralSolver/SpectralHankelTransform/HankelTransform.H
@@ -32,18 +32,18 @@ class HankelTransform
         using RealVector = amrex::Gpu::DeviceVector<amrex::Real>;
 
         // Constructor
-        HankelTransform(const int hankel_order,
-                        const int azimuthal_mode,
-                        const int nr,
-                        const amrex::Real rmax);
+        HankelTransform(int hankel_order,
+                        int azimuthal_mode,
+                        int nr,
+                        amrex::Real rmax);
 
         const RealVector & getSpectralWavenumbers() {return m_kr;}
 
-        void HankelForwardTransform(amrex::FArrayBox const& F, int const F_icomp,
-                                    amrex::FArrayBox      & G, int const G_icomp);
+        void HankelForwardTransform(amrex::FArrayBox const& F, int F_icomp,
+                                    amrex::FArrayBox      & G, int G_icomp);
 
-        void HankelInverseTransform(amrex::FArrayBox const& G, int const G_icomp,
-                                    amrex::FArrayBox      & F, int const F_icomp);
+        void HankelInverseTransform(amrex::FArrayBox const& G, int G_icomp,
+                                    amrex::FArrayBox      & F, int F_icomp);
 
     private:
         // Even though nk == nr always, use a separate variable for clarity.