From ab52b176f49c93b21e89710eb930cf55a91964d6 Mon Sep 17 00:00:00 2001
From: Ulf Hermann <ulf.hermann@qt.io>
Date: Fri, 4 Jun 2021 17:12:42 +0200
Subject: [PATCH] Avoid unnecessary work when linking observers

We already do most of the work of setFirstObserver() in the method body
before.

Change-Id: Ia31f19ca656675dddb692609d8875c5d48e967f1
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
---
 src/corelib/kernel/qproperty.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/corelib/kernel/qproperty.cpp b/src/corelib/kernel/qproperty.cpp
index 32c73278ec9..43fc84159a0 100644
--- a/src/corelib/kernel/qproperty.cpp
+++ b/src/corelib/kernel/qproperty.cpp
@@ -83,8 +83,8 @@ void QPropertyBindingDataPointer::addObserver(QPropertyObserver *observer)
         observer->next = firstObserver;
         if (observer->next)
             observer->next->prev = &observer->next;
+        d = reinterpret_cast<quintptr>(observer);
     }
-    setFirstObserver(observer);
 }
 
 /*!