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/Parallelization/WarpXRegrid.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/Parallelization/WarpXRegrid.cpp')
| -rw-r--r-- | Source/Parallelization/WarpXRegrid.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Source/Parallelization/WarpXRegrid.cpp b/Source/Parallelization/WarpXRegrid.cpp index 25eff3045..4d10b3691 100644 --- a/Source/Parallelization/WarpXRegrid.cpp +++ b/Source/Parallelization/WarpXRegrid.cpp @@ -292,7 +292,7 @@ WarpX::RemakeLevel (int lev, Real /*time*/, const BoxArray& ba, const Distributi if (spectral_solver_cp[lev] != nullptr) { BoxArray cba = ba; cba.coarsen(refRatio(lev-1)); - std::array<Real,3> cdx = CellSize(lev-1); + const std::array<Real,3> cdx = CellSize(lev-1); // Get the cell-centered box BoxArray c_realspace_ba = cba; // Copy box @@ -345,7 +345,7 @@ WarpX::RemakeLevel (int lev, Real /*time*/, const BoxArray& ba, const Distributi { costs[lev] = std::make_unique<LayoutData<Real>>(ba, dm); const auto iarr = costs[lev]->IndexArray(); - for (int i : iarr) + for (const auto& i : iarr) { (*costs[lev])[i] = 0.0; setLoadBalanceEfficiency(lev, -1); @@ -402,7 +402,7 @@ WarpX::ResetCosts () for (int lev = 0; lev <= finest_level; ++lev) { const auto iarr = costs[lev]->IndexArray(); - for (int i : iarr) + for (const auto& i : iarr) { // Reset costs (*costs[lev])[i] = 0.0; |