Skip to content

Commit

Permalink
a few more clang-tidy fixes (#2804)
Browse files Browse the repository at this point in the history
  • Loading branch information
zingale authored Mar 27, 2024
1 parent 760ecc4 commit aa8d3b2
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 2 deletions.
5 changes: 4 additions & 1 deletion Source/driver/Castro_advance_ctu.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,16 @@
using namespace amrex;

advance_status
Castro::do_advance_ctu (Real time, Real dt)
Castro::do_advance_ctu (Real time, Real dt) // NOLINT(readability-convert-member-functions-to-static)
{
// this routine will advance the old state data (called Sborder here)
// to the new time, for a single level. The new data is called
// S_new here. The update includes reactions (if we are not doing
// SDC), hydro, and the source terms.

amrex::ignore_unused(time);
amrex::ignore_unused(dt);

BL_PROFILE("Castro::do_advance_ctu()");

advance_status status {};
Expand Down
3 changes: 3 additions & 0 deletions Source/driver/Castro_advance_sdc.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ Castro::do_advance_sdc (Real time,

// this is the new "formal" SDC integration routine.

amrex::ignore_unused(amr_iteration);
amrex::ignore_unused(amr_ncycle);

// this does the entire update in time for 1 SDC iteration.

BL_PROFILE("Castro::do_advance_sdc()");
Expand Down
5 changes: 4 additions & 1 deletion Source/hydro/Castro_ctu_hydro.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,11 @@
using namespace amrex;

advance_status
Castro::construct_ctu_hydro_source(Real time, Real dt)
Castro::construct_ctu_hydro_source(Real time, Real dt) // NOLINT(readability-convert-member-functions-to-static)
{
amrex::ignore_unused(time);
amrex::ignore_unused(dt);

advance_status status {};

if (!do_hydro) {
Expand Down
4 changes: 4 additions & 0 deletions Source/hydro/Castro_mol_hydro.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ void
Castro::construct_mol_hydro_source(Real time, Real dt, MultiFab& A_update)
{

amrex::ignore_unused(time);

#ifdef RADIATION
amrex::Abort("Error: radiation not supported for the MOL hydro source term");
#else
Expand Down Expand Up @@ -648,6 +650,8 @@ Castro::construct_mol_hydro_source(Real time, Real dt, MultiFab& A_update)
}

Array4<Real> pradial_fab = pradial.array();
#endif
#if AMREX_SPACEDIM == 1
Array4<Real> const qex_arr = qe[0].array();
#endif

Expand Down
3 changes: 3 additions & 0 deletions Source/hydro/advection_util.H
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ void src_to_prim(int i, int j, int k, const Real dt,
#endif
Array4<Real> const& srcQ)
{

amrex::ignore_unused(dt);

for (int n = 0; n < NQSRC; ++n) {
srcQ(i,j,k,n) = 0.0_rt;
}
Expand Down

0 comments on commit aa8d3b2

Please sign in to comment.