From 4722571a5ad0f1c6afe68aaecd4f864618fd59d2 Mon Sep 17 00:00:00 2001 From: wmayer Date: Fri, 3 Jul 2020 15:49:32 +0200 Subject: [PATCH] Surface: optimize Extend::onChanged --- src/Mod/Surface/App/FeatureExtend.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/Mod/Surface/App/FeatureExtend.cpp b/src/Mod/Surface/App/FeatureExtend.cpp index 8abbd3a193..53257681de 100644 --- a/src/Mod/Surface/App/FeatureExtend.cpp +++ b/src/Mod/Surface/App/FeatureExtend.cpp @@ -161,10 +161,9 @@ void Extend::onChanged(const App::Property* prop) lockOnChangeMutex.unlock(); std::lock_guard lock(lockOnChangeMutex); - if ( ExtendUSymetric.getValue() ) + if (ExtendUSymetric.getValue()) { - if (prop->getName() == ExtendUNeg.getName() - || prop->getName() == ExtendUPos.getName()) + if (prop == &ExtendUNeg || prop == &ExtendUPos) { auto changedValue = dynamic_cast(prop); if (changedValue) @@ -174,10 +173,10 @@ void Extend::onChanged(const App::Property* prop) } } } + if (ExtendVSymetric.getValue()) { - if (prop->getName() == ExtendVNeg.getName() - || prop->getName() == ExtendVPos.getName()) + if (prop == &ExtendVNeg || prop == &ExtendVPos) { auto changedValue = dynamic_cast(prop); if (changedValue)