Separate the error case when evaluating bindings

There's no point in returning a usually empty error when
evaluating bindings, adding overhead to the regular code
path.

Instead, the error can be set on the currently evaluating
binding if required. This streamlines the functor used to
wrap the binding and should thus expand to less code and
execute faster in the regular case.

To achieve this, expose a pointer to the currently evaluating
binding in the private API (as QtQml needs it to be able to
report errors).

The error case now requires one additional TLS lookup, but
we don't really care about performance in that case anyway.

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

1 files changed, 5 insertions, 0 deletions

diff --git a/src/corelib/kernel/qproperty.cpp b/src/corelib/kernel/qproperty.cpp
index 92c9afc6d8..f7039cc62a 100644
--- a/src/corelib/kernel/qproperty.cpp
+++ b/src/corelib/kernel/qproperty.cpp
@@ -206,6 +206,11 @@ BindingEvaluationState::~BindingEvaluationState()
     *currentState = previousState;
 }
 
+QPropertyBindingPrivate *QPropertyBindingPrivate::currentlyEvaluatingBinding()
+{
+    return currentBindingEvaluationState ? currentBindingEvaluationState->binding : nullptr;
+}
+
 void QPropertyBase::evaluateIfDirty()
 {
     QPropertyBasePointer d{this};