diff options
Diffstat (limited to 'al/effects/echo.cpp')
-rw-r--r-- | al/effects/echo.cpp | 61 |
1 files changed, 14 insertions, 47 deletions
diff --git a/al/effects/echo.cpp b/al/effects/echo.cpp index 98fa5541..3a83b24f 100644 --- a/al/effects/echo.cpp +++ b/al/effects/echo.cpp @@ -207,53 +207,20 @@ bool EchoCommitter::commit(const EaxEffectProps &props) const auto orig = props_; props_ = props; - auto is_dirty = bool{orig.mType != props_.mType}; - if(props_.mEcho.flDelay != props.mEcho.flDelay) - { - is_dirty = true; - al_effect_props_.Echo.Delay = clamp( - props_.mEcho.flDelay, - AL_ECHO_MIN_DELAY, - AL_ECHO_MAX_DELAY); - } - - if(props_.mEcho.flLRDelay != props.mEcho.flLRDelay) - { - is_dirty = true; - al_effect_props_.Echo.LRDelay = clamp( - props_.mEcho.flLRDelay, - AL_ECHO_MIN_LRDELAY, - AL_ECHO_MAX_LRDELAY); - } - - if(props_.mEcho.flDamping != props.mEcho.flDamping) - { - is_dirty = true; - al_effect_props_.Echo.Damping = clamp( - props_.mEcho.flDamping, - AL_ECHO_MIN_DAMPING, - AL_ECHO_MAX_DAMPING); - } - - if(props_.mEcho.flFeedback != props.mEcho.flFeedback) - { - is_dirty = true; - al_effect_props_.Echo.Feedback = clamp( - props_.mEcho.flFeedback, - AL_ECHO_MIN_FEEDBACK, - AL_ECHO_MAX_FEEDBACK); - } - - if(props_.mEcho.flSpread != props.mEcho.flSpread) - { - is_dirty = true; - al_effect_props_.Echo.Spread = clamp( - props_.mEcho.flSpread, - AL_ECHO_MIN_SPREAD, - AL_ECHO_MAX_SPREAD); - } - - return is_dirty; + if(orig.mType == props_.mType && props_.mEcho.flDelay == props.mEcho.flDelay + && props_.mEcho.flLRDelay == props.mEcho.flLRDelay + && props_.mEcho.flDamping == props.mEcho.flDamping + && props_.mEcho.flFeedback == props.mEcho.flFeedback + && props_.mEcho.flSpread == props.mEcho.flSpread) + return false; + + al_effect_props_.Echo.Delay = props_.mEcho.flDelay; + al_effect_props_.Echo.LRDelay = props_.mEcho.flLRDelay; + al_effect_props_.Echo.Damping = props_.mEcho.flDamping; + al_effect_props_.Echo.Feedback = props_.mEcho.flFeedback; + al_effect_props_.Echo.Spread = props_.mEcho.flSpread; + + return true; } template<> |