QProperty: Support multiple observers

Previously, only the first observer would get notified. Also, make sure
that the notifiers are always retained when switching between bindings
and values.

Change-Id: I9c25c0f2e288dac3a335b68e618f7ddeb44be25a
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/corelib/kernel/qproperty.h b/src/corelib/kernel/qproperty.h
index 22313bc92e..938de61fd5 100644
--- a/src/corelib/kernel/qproperty.h
+++ b/src/corelib/kernel/qproperty.h
@@ -254,16 +254,16 @@ public:
 
     void setValue(T &&newValue)
     {
+        d.priv.removeBinding();
         if (d.setValueAndReturnTrueIfChanged(std::move(newValue)))
             notify();
-        d.priv.removeBinding();
     }
 
     void setValue(const T &newValue)
     {
+        d.priv.removeBinding();
         if (d.setValueAndReturnTrueIfChanged(newValue))
             notify();
-        d.priv.removeBinding();
     }
 
     QProperty<T> &operator=(T &&newValue)