diff options
| author |  Luca Fedeli <luca.fedeli@cea.fr>
| 2023-06-13 05:07:42 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2023-06-13 03:07:42 +0000 |
| commit | 297dd2d31527acc9676d4eb0c3a9c54c2688abd6 (patch) | |
| tree | 86e14ea3325872030f7fe3fb7b6e20013fa1d032 /Source/Particles/RigidInjectedParticleContainer.cpp | |
| parent | d1a2d04521691a9489f53cc003142e91d98571ee (diff) | |
| download | WarpX-297dd2d31527acc9676d4eb0c3a9c54c2688abd6.tar.gz WarpX-297dd2d31527acc9676d4eb0c3a9c54c2688abd6.tar.zst WarpX-297dd2d31527acc9676d4eb0c3a9c54c2688abd6.zip | |
Enforce const correctness using clang-tidy CI test (#3921)
* add clang-tidy workflow
* fix missing newline
* [pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
* make clang.14.sh executable
* remove non explicitly selected checks
* complete list of dependencies
* fix bug
* specify path of clang-tidy file
* fix bug
* add new check
* remove one check
* add magic numbers check
* removed one check
* keep only one check
* Docs: Conda Dev w/ Boost (#3911)
Add the `boost` package to the conda developer environment.
Used for QED table generation.
* Fix typo in Adastra cluster documentation (#3918)
* add back three checks
* prepare clang-tidy wrapper
* actually use clang-tidy in the script
* test
* fix bug
* actually use clang-tidy
* fixed bug
* fixed bug
* fixed bug
* fixed bug
* remove all checks except the selected 3
* fixed bug
* fixed bug
* fixed bug
* enforce const correctness using clang-tidy
* remove one check
* Fix Concurrency Issue
* BLAS++/LAPACK++ for RZ+PSATD
* Build all 4 WarpX DIMS
* add few echo for test purposes
* try to debug mysterious error
* don't test RZ with clang-tidy
* add back RZ test
* add some const
* fix bug
* check also header files
* remove header filter since it does not work as expected
* fixed bug
* check also WarpX headers
* fix bugs
* continue enforcing const correctness
* continue enforcing const correctness
* [pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
* fixed bug
* fix bugs and add new const
* clean .clang-tidy
* make regex more precise according to Weiqun's suggestion
* add more const
* fix bugs
* fix bug
* silence warning on float comparison
* fixed bug
* fixed bugs
* fix bug and add const
* fixed bugs
* fix bug
* fix bug
* fix bug
* fixed bug
* fix bug
Diffstat (limited to 'Source/Particles/RigidInjectedParticleContainer.cpp')
| -rw-r--r-- | Source/Particles/RigidInjectedParticleContainer.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/Source/Particles/RigidInjectedParticleContainer.cpp b/Source/Particles/RigidInjectedParticleContainer.cpp index 8efe724bd..6ff621538 100644 --- a/Source/Particles/RigidInjectedParticleContainer.cpp +++ b/Source/Particles/RigidInjectedParticleContainer.cpp @@ -61,7 +61,7 @@ RigidInjectedParticleContainer::RigidInjectedParticleContainer (AmrCore* amr_cor : PhysicalParticleContainer(amr_core, ispecies, name) { - ParmParse pp_species_name(species_name); + const ParmParse pp_species_name(species_name); utils::parser::getWithParser( pp_species_name, "zinject_plane", zinject_plane); @@ -361,12 +361,12 @@ RigidInjectedParticleContainer::PushP (int lev, Real dt, const Dim3 lo = lbound(box); - bool galerkin_interpolation = WarpX::galerkin_interpolation; - int nox = WarpX::nox; - int n_rz_azimuthal_modes = WarpX::n_rz_azimuthal_modes; + const bool galerkin_interpolation = WarpX::galerkin_interpolation; + const int nox = WarpX::nox; + const int n_rz_azimuthal_modes = WarpX::n_rz_azimuthal_modes; - amrex::GpuArray<amrex::Real, 3> dx_arr = {dx[0], dx[1], dx[2]}; - amrex::GpuArray<amrex::Real, 3> xyzmin_arr = {xyzmin[0], xyzmin[1], xyzmin[2]}; + const amrex::GpuArray<amrex::Real, 3> dx_arr = {dx[0], dx[1], dx[2]}; + const amrex::GpuArray<amrex::Real, 3> xyzmin_arr = {xyzmin[0], xyzmin[1], xyzmin[2]}; amrex::Array4<const amrex::Real> const& ex_arr = exfab.array(); amrex::Array4<const amrex::Real> const& ey_arr = eyfab.array(); @@ -409,7 +409,7 @@ RigidInjectedParticleContainer::PushP (int lev, Real dt, enum exteb_flags : int { no_exteb, has_exteb }; - int exteb_runtime_flag = getExternalEB.isNoOp() ? no_exteb : has_exteb; + const int exteb_runtime_flag = getExternalEB.isNoOp() ? no_exteb : has_exteb; amrex::ParallelFor(TypeList<CompileTimeOptions<no_exteb,has_exteb>>{}, {exteb_runtime_flag}, |