Skip to content

Commit

Permalink
fix mismatch between fill_hybrid_hydro_source prototype and implement…
Browse files Browse the repository at this point in the history
…ation (#2781)

we have arguments swapped
also mark the incoming state const
  • Loading branch information
zingale authored Mar 18, 2024
1 parent 2e8fb11 commit 2893e8d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Source/hydro/Castro_hybrid.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ Castro::construct_new_hybrid_source(MultiFab& source, MultiFab& state_old, Multi


void
Castro::fill_hybrid_hydro_source(MultiFab& sources, MultiFab& state_in, Real mult_factor)
Castro::fill_hybrid_hydro_source(MultiFab& sources, const MultiFab& state_in, Real mult_factor)
{
BL_PROFILE("Castro::fill_hybrid_hydro_source()");

Expand All @@ -97,7 +97,7 @@ Castro::fill_hybrid_hydro_source(MultiFab& sources, MultiFab& state_in, Real mul
{
const Box& bx = mfi.tilebox();

auto u = state_in.array(mfi);
const auto u = state_in.array(mfi);
auto src = sources.array(mfi);

amrex::ParallelFor(bx,
Expand Down
2 changes: 1 addition & 1 deletion Source/hydro/Castro_hydro.H
Original file line number Diff line number Diff line change
Expand Up @@ -856,7 +856,7 @@
/// @param source MultiFab to save source to
/// @param mult_factor
///
void fill_hybrid_hydro_source(amrex::MultiFab& state, amrex::MultiFab& source, const amrex::Real mult_factor);
void fill_hybrid_hydro_source(amrex::MultiFab& source, const amrex::MultiFab& state, const amrex::Real mult_factor);


///
Expand Down

0 comments on commit 2893e8d

Please sign in to comment.