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/FieldSolver/SpectralSolver/SpectralKSpaceRZ.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/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp')
| -rw-r--r-- | Source/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Source/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp b/Source/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp index 4b02d1b7e..18e46d676 100644 --- a/Source/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp +++ b/Source/FieldSolver/SpectralSolver/SpectralKSpaceRZ.cpp @@ -35,11 +35,11 @@ SpectralKSpaceRZ::SpectralKSpaceRZ (const amrex::BoxArray& realspace_ba, // For local FFTs, boxes in spectral space start at 0 in // each direction and have the same number of points as the // (cell-centered) real space box - amrex::Box realspace_bx = realspace_ba[i]; - amrex::IntVect fft_size = realspace_bx.length(); - amrex::IntVect spectral_bx_size = fft_size; + const amrex::Box realspace_bx = realspace_ba[i]; + const amrex::IntVect fft_size = realspace_bx.length(); + const amrex::IntVect spectral_bx_size = fft_size; // Define the corresponding box - amrex::Box spectral_bx = amrex::Box(amrex::IntVect::TheZeroVector(), + const amrex::Box spectral_bx = amrex::Box(amrex::IntVect::TheZeroVector(), spectral_bx_size - amrex::IntVect::TheUnitVector() ); spectral_bl.push_back(spectral_bx); } |