Skip to content

Commit

Permalink
Merge pull request #426 from weefuzzy/fix/load_paramupdate
Browse files Browse the repository at this point in the history
For `load` and `read` only update params on success :grimace:
  • Loading branch information
weefuzzy authored Sep 13, 2024
2 parents c5c5c87 + f88f6f1 commit 4aa49e0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion source/include/FluidMaxWrapper.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -2111,7 +2111,8 @@ class FluidMaxWrapper
updateParams(FluidMaxWrapper* x,
MessageResult<typename ParamSetType::ValueTuple> const& v)
{
x->mParams.fromTuple(typename ParamSetType::ValueTuple(v.value()));
if(v.ok())
x->mParams.fromTuple(typename ParamSetType::ValueTuple(v.value()));
}

static void updateParams(FluidMaxWrapper*, MessageResult<void>) {}
Expand Down

0 comments on commit 4aa49e0

Please sign in to comment.