diff options
| author |  Luca Fedeli <luca.fedeli@cea.fr>
| 2023-08-01 13:38:19 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2023-08-01 13:38:19 +0200 |
| commit | 0aa12461bd956e9e9a08e0ac3b8e2c2263aa7661 (patch) | |
| tree | 3a96aabdc6ad518c2d1300800aaa8ee2cafebba3 /Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp | |
| parent | 63dc76ff15c1c8660f30e1ff1cfcb9c2f04db814 (diff) | |
| download | WarpX-0aa12461bd956e9e9a08e0ac3b8e2c2263aa7661.tar.gz WarpX-0aa12461bd956e9e9a08e0ac3b8e2c2263aa7661.tar.zst WarpX-0aa12461bd956e9e9a08e0ac3b8e2c2263aa7661.zip | |
Clang tidy CI test: add several readability checks to clang tidy CI test (#4124)
* add few readability checks to clang-tidy CI test
* address all the issues found with clang-tidy
* fix bug
* fixed bug
* fix residual issue
* fix issue found with clang-tidy
Diffstat (limited to 'Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp')
| -rw-r--r-- | Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp b/Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp index ab4e39710..0cd7a3463 100644 --- a/Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp +++ b/Source/FieldSolver/SpectralSolver/SpectralFieldData.cpp @@ -47,7 +47,7 @@ SpectralFieldIndex::SpectralFieldIndex (const bool update_with_rho, int c = 0; - if (pml == false) + if (!pml) { Ex = c++; Ey = c++; Ez = c++; Bx = c++; By = c++; Bz = c++; @@ -291,15 +291,15 @@ SpectralFieldData::ForwardTransform (const int lev, Complex spectral_field_value = tmp_arr(i,j,k); // Apply proper shift in each dimension #if (AMREX_SPACEDIM >= 2) - if (is_nodal_x==false) spectral_field_value *= xshift_arr[i]; + if (!is_nodal_x) spectral_field_value *= xshift_arr[i]; #endif #if defined(WARPX_DIM_3D) - if (is_nodal_y==false) spectral_field_value *= yshift_arr[j]; - if (is_nodal_z==false) spectral_field_value *= zshift_arr[k]; + if (!is_nodal_y) spectral_field_value *= yshift_arr[j]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[k]; #elif defined(WARPX_DIM_XZ) || defined(WARPX_DIM_RZ) - if (is_nodal_z==false) spectral_field_value *= zshift_arr[j]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[j]; #elif defined(WARPX_DIM_1D_Z) - if (is_nodal_z==false) spectral_field_value *= zshift_arr[i]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[i]; #endif // Copy field into the right index fields_arr(i,j,k,field_index) = spectral_field_value; @@ -391,15 +391,15 @@ SpectralFieldData::BackwardTransform (const int lev, Complex spectral_field_value = field_arr(i,j,k,field_index); // Apply proper shift in each dimension #if (AMREX_SPACEDIM >= 2) - if (is_nodal_x==false) spectral_field_value *= xshift_arr[i]; + if (!is_nodal_x) spectral_field_value *= xshift_arr[i]; #endif #if defined(WARPX_DIM_3D) - if (is_nodal_y==false) spectral_field_value *= yshift_arr[j]; - if (is_nodal_z==false) spectral_field_value *= zshift_arr[k]; + if (!is_nodal_y) spectral_field_value *= yshift_arr[j]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[k]; #elif defined(WARPX_DIM_XZ) || defined(WARPX_DIM_RZ) - if (is_nodal_z==false) spectral_field_value *= zshift_arr[j]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[j]; #elif defined(WARPX_DIM_1D_Z) - if (is_nodal_z==false) spectral_field_value *= zshift_arr[i]; + if (!is_nodal_z) spectral_field_value *= zshift_arr[i]; #endif // Copy field into temporary array tmp_arr(i,j,k) = spectral_field_value; @@ -444,11 +444,11 @@ SpectralFieldData::BackwardTransform (const int lev, #endif // If necessary, do not fill the guard cells // (shrink box by passing negative number of cells) - if (m_periodic_single_box == false) + if (!m_periodic_single_box) { for (int dir = 0; dir < AMREX_SPACEDIM; dir++) { - if (static_cast<bool>(fill_guards[dir]) == false) mf_box.grow(dir, -mf_ng[dir]); + if ((fill_guards[dir]) == 0) mf_box.grow(dir, -mf_ng[dir]); } } |