From 56316d6c12f74707f5d32e14a048527551735816 Mon Sep 17 00:00:00 2001 From: "Eunki, Hong" Date: Mon, 19 Dec 2022 15:59:13 +0900 Subject: [PATCH] Reduce GetCurrentSize call time Call mOwner.GetCurrentSize() only if we use it Change-Id: I1a22b4d86ad97856a14aa367f1b3d4b62b7e31bf Signed-off-by: Eunki, Hong --- dali/internal/event/actors/actor-sizer.cpp | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/dali/internal/event/actors/actor-sizer.cpp b/dali/internal/event/actors/actor-sizer.cpp index 4d8e6d2..81e3f4d 100644 --- a/dali/internal/event/actors/actor-sizer.cpp +++ b/dali/internal/event/actors/actor-sizer.cpp @@ -117,14 +117,7 @@ void ActorSizer::SetSizeInternal(const Vector3& size) // dont allow recursive loop DALI_ASSERT_ALWAYS(!mInsideOnSizeSet && "Cannot call SetSize from OnSizeSet"); // check that we have a node AND the new size width, height or depth is at least a little bit different from the old one - Vector3 currentSize = mOwner.GetCurrentSize(); - - if((fabsf(mTargetSize.width - size.width) > Math::MACHINE_EPSILON_1) || - (fabsf(mTargetSize.height - size.height) > Math::MACHINE_EPSILON_1) || - (fabsf(mTargetSize.depth - size.depth) > Math::MACHINE_EPSILON_1) || - (fabsf(mTargetSize.width - currentSize.width) > Math::MACHINE_EPSILON_1) || - (fabsf(mTargetSize.height - currentSize.height) > Math::MACHINE_EPSILON_1) || - (fabsf(mTargetSize.depth - currentSize.depth) > Math::MACHINE_EPSILON_1)) + if(mTargetSize != size || mTargetSize != mOwner.GetCurrentSize()) { mTargetSize = size; -- 2.7.4