Skip to content

Use more consistently const reference in SolverGodunovFiveEq. #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pkestene merged 1 commit into from
Jul 26, 2026
Merged

Use more consistently const reference in SolverGodunovFiveEq. #21

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions SolverGodunovFiveEq.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -987,7 +987,7 @@ SolverGodunovFiveEq<dim, device_t>::save_solution_hdf5([[maybe_unused]] bool pur
template <size_t dim, typename device_t>
void
SolverGodunovFiveEq<dim, device_t>::synchronize_mpi_ghost_data(
[[maybe_unused]] DataArrayBlock_t data)
[[maybe_unused]] DataArrayBlock_t const & data)
{

#ifdef KALYPSSO_CORE_USE_MPI
Expand All @@ -1004,7 +1004,7 @@ SolverGodunovFiveEq<dim, device_t>::synchronize_mpi_ghost_data(
// =======================================================
template <size_t dim, typename device_t>
void
SolverGodunovFiveEq<dim, device_t>::fill_outside_quadrants(DataArrayBlock_t data)
SolverGodunovFiveEq<dim, device_t>::fill_outside_quadrants(DataArrayBlock_t const & data)
{

// assert data has expected size (in terms of number of octants/quadrants)
Expand Down
4 changes: 2 additions & 2 deletions SolverGodunovFiveEq.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -419,7 +419,7 @@ class SolverGodunovFiveEq : public kalypsso::SolverBase
//! synchronize MPI ghost data / only necessary when MPI is activated
//! this will also do external border only when mesh is periodic
void
synchronize_mpi_ghost_data(DataArrayBlock_t data);
synchronize_mpi_ghost_data(DataArrayBlock_t const & data);

//! when mesh is not periodic, the number of outside quadrants is not zero, so we need to fill
//! outside quadrant user data according to the border condition. This is where we apply border
Expand All @@ -428,7 +428,7 @@ class SolverGodunovFiveEq : public kalypsso::SolverBase
//! Just to be clear, let's repeat that array m_U must be sized upon the total number of
//! octants (owned, ghost and outside).
void
fill_outside_quadrants(DataArrayBlock_t data);
fill_outside_quadrants(DataArrayBlock_t const & data);

//! mark cells for refinement
void
Expand Down
Loading