diff options
| author | Ivan Solovev <ivan.solovev@qt.io> | 2023-08-25 17:21:11 +0200 |
|---|---|---|
| committer | Ivan Solovev <ivan.solovev@qt.io> | 2023-09-03 18:20:18 +0200 |
| commit | b64e36240b807e6dba783732593036439fec8a62 (patch) | |
| tree | f9f0ab2e300507f7c2882a84daf5c7cd929014ba /src/corelib/animation/qpauseanimation.cpp | |
| parent | df9651c3fcdd026255add31744b2f75dc2ecdd62 (diff) | |
QPauseAnimation: extend tests and fix binding loop
Extend the unit-tests for bindable properties and fix the discovered
binding loop by using {set}ValueBypassingBindings() in the setter of
the duration property.
The code refactoring does not modify the setter logic, because
previously the binding was anyway implicitly removed when calling the
assignment operator. The updated code just does it explicitly.
Task-number: QTBUG-116346
Pick-to: 6.6 6.5
Change-Id: I0f339d182efb60500ee7f12e407f200d739da312
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
Diffstat (limited to 'src/corelib/animation/qpauseanimation.cpp')
| -rw-r--r-- | src/corelib/animation/qpauseanimation.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/corelib/animation/qpauseanimation.cpp b/src/corelib/animation/qpauseanimation.cpp index eb1e6c3c81b..344b21946e3 100644 --- a/src/corelib/animation/qpauseanimation.cpp +++ b/src/corelib/animation/qpauseanimation.cpp @@ -94,11 +94,10 @@ void QPauseAnimation::setDuration(int msecs) } Q_D(QPauseAnimation); - if (msecs != d->duration) { - d->duration = msecs; + d->duration.removeBindingUnlessInWrapper(); + if (msecs != d->duration.valueBypassingBindings()) { + d->duration.setValueBypassingBindings(msecs); d->duration.notify(); - } else { - d->duration.removeBindingUnlessInWrapper(); } } |