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/Utils/MPIInitHelpers.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/Utils/MPIInitHelpers.cpp')
| -rw-r--r-- | Source/Utils/MPIInitHelpers.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/Source/Utils/MPIInitHelpers.cpp b/Source/Utils/MPIInitHelpers.cpp index 47411375a..9d205eb5f 100644 --- a/Source/Utils/MPIInitHelpers.cpp +++ b/Source/Utils/MPIInitHelpers.cpp @@ -41,12 +41,13 @@ namespace utils std::pair< int, int > warpx_mpi_init (int argc, char* argv[]) { - int thread_required = warpx_mpi_thread_required(); - int thread_provided = -1; + const int thread_required = warpx_mpi_thread_required(); #ifdef AMREX_USE_MPI + int thread_provided = -1; MPI_Init_thread(&argc, &argv, thread_required, &thread_provided); #else amrex::ignore_unused(argc, argv); + const int thread_provided = -1; #endif return std::make_pair(thread_required, thread_provided); } @@ -64,7 +65,7 @@ namespace utils warpx_check_mpi_thread_level () { #ifdef AMREX_USE_MPI - int thread_required = warpx_mpi_thread_required(); + const int thread_required = warpx_mpi_thread_required(); int thread_provided = -1; MPI_Query_thread(&thread_provided); auto mtn = amrex::ParallelDescriptor::mpi_level_to_string; |