Skip to content

Commit

Permalink
explicit to solve new reported code smell on constructor of DummyStra…
Browse files Browse the repository at this point in the history
…tegyWrapper.
  • Loading branch information
WPK4FEM committed Oct 4, 2023
1 parent 8ecb1f4 commit e4e2ecf
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class ProcessSpy : public Process
class DummyStrategyWrapper : public StrategyWrapper
{
public:
DummyStrategyWrapper::DummyStrategyWrapper(bool convergence_needed) :
explicit DummyStrategyWrapper::DummyStrategyWrapper(bool convergence_needed) :
mConvergenceNeeded ( convergence_needed ? TimeStepEndState::ConvergenceState::converged : TimeStepEndState::ConvergenceState::non_converged) {}
[[nodiscard]] TimeStepEndState::ConvergenceState GetConvergenceState(const TimeStepEndState& rEndState) override { return mConvergenceNeeded;};
[[nodiscard]] std::size_t GetNumberOfIterations() const override { return 4;};
Expand Down

0 comments on commit e4e2ecf

Please sign in to comment.