diff options
author | Antti Määttä <antti.maatta@qt.io> | 2020-01-20 08:38:32 +0200 |
---|---|---|
committer | Antti Määttä <antti.maatta@qt.io> | 2020-01-20 11:31:05 +0200 |
commit | 64f4a0f31565dce726d1306c0e36dea517bf80f3 (patch) | |
tree | be4e802d5a374571c9b4825de7e40a564f201f4e | |
parent | 29eab4197f6cd2d6725c71bc612356c43f872993 (diff) |
Remove unused and duplicate code
-Remove tree traverse functions from actimation manager.
-Remove duplicate element helper code.
Change-Id: Ibf23c58755024a7374b8279861e4aad8fbcdd7c7
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
-rw-r--r-- | src/engine/Qt3DSRenderRuntimeBindingImplTranslation.cpp | 4 | ||||
-rw-r--r-- | src/ogl-runtime-static/ogl-runtime-static.pro | 2 | ||||
-rw-r--r-- | src/runtime/Qt3DSActivationManager.cpp | 15 | ||||
-rw-r--r-- | src/runtime/Qt3DSElementHelper.cpp | 141 | ||||
-rw-r--r-- | src/runtime/Qt3DSElementHelper.h | 57 |
5 files changed, 2 insertions, 217 deletions
diff --git a/src/engine/Qt3DSRenderRuntimeBindingImplTranslation.cpp b/src/engine/Qt3DSRenderRuntimeBindingImplTranslation.cpp index f4b0f31..bdf92e5 100644 --- a/src/engine/Qt3DSRenderRuntimeBindingImplTranslation.cpp +++ b/src/engine/Qt3DSRenderRuntimeBindingImplTranslation.cpp @@ -37,7 +37,7 @@ #include "Qt3DSHash.h" #include "Qt3DSRenderPlugin.h" #include "Qt3DSRenderPluginPropertyValue.h" -#include "Qt3DSElementHelper.h" +#include "Qt3DSQmlElementHelper.h" #include "Qt3DSPresentation.h" #include "Qt3DSApplication.h" #include "Qt3DSRenderCustomMaterialSystem.h" @@ -191,7 +191,7 @@ struct SRuntimePropertyParser } else if (m_Type == Q3DStudio::ATTRIBUTETYPE_STRING) { CRegisteredString theString = m_RenderContext.GetStringTable().HandleToStr(m_Value.m_StringHandle); - theElem = Q3DStudio::CElementHelper::GetElement( + theElem = Q3DStudio::CQmlElementHelper::GetElement( m_Element.GetBelongedPresentation()->GetApplication(), m_Element.GetBelongedPresentation(), theString.c_str(), &m_Element); } diff --git a/src/ogl-runtime-static/ogl-runtime-static.pro b/src/ogl-runtime-static/ogl-runtime-static.pro index e2480f2..ca90711 100644 --- a/src/ogl-runtime-static/ogl-runtime-static.pro +++ b/src/ogl-runtime-static/ogl-runtime-static.pro @@ -58,7 +58,6 @@ SOURCES += \ ../runtime/Qt3DSInputEngine.cpp \ ../runtime/Qt3DSLogicSystem.cpp \ ../runtime/Qt3DSCommandHelper.cpp \ - ../runtime/Qt3DSElementHelper.cpp \ ../runtime/Qt3DSOutputMemoryStream.cpp \ ../runtime/Qt3DSParametersSystem.cpp \ ../runtime/Qt3DSPresentation.cpp \ @@ -251,7 +250,6 @@ HEADERS += \ ../runtime/Qt3DSInputEngine.h \ ../runtime/Qt3DSLogicSystem.h \ ../runtime/Qt3DSCommandHelper.h \ - ../runtime/Qt3DSElementHelper.h \ ../runtime/Qt3DSOutputMemoryStream.h \ ../runtime/Qt3DSParametersSystem.h \ ../runtime/Qt3DSPresentation.h \ diff --git a/src/runtime/Qt3DSActivationManager.cpp b/src/runtime/Qt3DSActivationManager.cpp index 784f42e..9ee77ae 100644 --- a/src/runtime/Qt3DSActivationManager.cpp +++ b/src/runtime/Qt3DSActivationManager.cpp @@ -107,21 +107,6 @@ typedef nvvector<STimeEvent *> TTimeEventList; struct STimeContext; typedef NVDataRef<NVScopedRefCounted<STimeContext>> TTimeContextSet; -// Tree navigation - -SElement *GetElementNodeFirstChild(SElement &inNode) -{ - return inNode.m_Child; -} -SElement *GetElementNodeNextSibling(SElement &inNode) -{ - return inNode.m_Sibling; -} -SElement *GetElementNodeParent(SElement &inNode) -{ - return inNode.m_Parent; -} - struct SScanBufferEntry { SElement *m_Node; diff --git a/src/runtime/Qt3DSElementHelper.cpp b/src/runtime/Qt3DSElementHelper.cpp deleted file mode 100644 index 4bab37b..0000000 --- a/src/runtime/Qt3DSElementHelper.cpp +++ /dev/null @@ -1,141 +0,0 @@ -/**************************************************************************** -** -** Copyright (C) 1993-2009 NVIDIA Corporation. -** Copyright (C) 2017 The Qt Company Ltd. -** Contact: https://www.qt.io/licensing/ -** -** This file is part of Qt 3D Studio. -** -** $QT_BEGIN_LICENSE:GPL$ -** Commercial License Usage -** Licensees holding valid commercial Qt licenses may use this file in -** accordance with the commercial license agreement provided with the -** Software or, alternatively, in accordance with the terms contained in -** a written agreement between you and The Qt Company. For licensing terms -** and conditions see https://www.qt.io/terms-conditions. For further -** information use the contact form at https://www.qt.io/contact-us. -** -** GNU General Public License Usage -** Alternatively, this file may be used under the terms of the GNU -** General Public License version 3 or (at your option) any later version -** approved by the KDE Free Qt Foundation. The licenses are as published by -** the Free Software Foundation and appearing in the file LICENSE.GPL3 -** included in the packaging of this file. Please review the following -** information to ensure the GNU General Public License requirements will -** be met: https://www.gnu.org/licenses/gpl-3.0.html. -** -** $QT_END_LICENSE$ -** -****************************************************************************/ - -#include "RuntimePrefix.h" - -#include "Qt3DSElementHelper.h" -#include "Qt3DSPresentation.h" -#include "Qt3DSApplication.h" -#include "Qt3DSCommandEventTypes.h" - -using namespace qt3ds; - -namespace Q3DStudio { - -const char PRESENTATION_DELIMITER = ':'; -const char NODE_DELIMITER = '.'; -const TStringHash RESERVED_THIS = CHash::HashString("this"); -const TStringHash RESERVED_PARENT = CHash::HashString("parent"); -const TStringHash RESERVED_SCENE = CHash::HashString("Scene"); - -CElementHelper::CElementHelper() -{ -} - -CElementHelper::~CElementHelper() -{ -} - -TElement *CElementHelper::GetElement(qt3ds::runtime::IApplication &inApplication, - IPresentation *inDefaultPresentation, const char *inPath, - TElement *inStartElement) -{ - if (!inPath || *inPath == 0) - return nullptr; - const char *thePath(inPath); - const char *theSubPath = nullptr; - IPresentation *thePresentation = nullptr; - size_t thePathLength = ::strlen(thePath) + 1; - char *theToken = Q3DStudio_allocate_desc(CHAR, thePathLength, "Token:TempPath"); - // Try to get the specified presentation - theSubPath = ::strchr(thePath, PRESENTATION_DELIMITER); - TElement *theElement = inStartElement; - if (theSubPath) { - UINT32 theSubPathLength = static_cast<UINT32>(theSubPath - thePath); - - ::strncpy(theToken, thePath, theSubPathLength); - theToken[theSubPathLength] = '\0'; - - thePath = theSubPath + 1; - - const CHAR *thePresentationName = theToken; - - thePresentation = inApplication.LoadAndGetPresentationById(thePresentationName); - } - - if (!thePresentation) - thePresentation = inDefaultPresentation; - - // Return nil if the inStartElement is not in the specified presentation - if (theElement - && (!theSubPath && theElement->GetBelongedPresentation() != thePresentation)) { - thePresentation = theElement->GetBelongedPresentation(); - } - - if (!thePresentation) - return nullptr; - - TStringHash theName; - INT32 theParseCounter = 0; - - while (thePath && thePath[0] != '\0') { - ++theParseCounter; - - // Do some strtok() work here - theSubPath = ::strchr(thePath, NODE_DELIMITER); - if (theSubPath) { - UINT32 theSubPathLength = static_cast<UINT32>(theSubPath - thePath); - Q3DStudio_ASSERT(theSubPathLength < thePathLength); - - ::strncpy(theToken, thePath, theSubPathLength); - theToken[theSubPathLength] = '\0'; - - thePath = theSubPath + 1; - } else { - ::strcpy(theToken, thePath); - thePath = nullptr; - } - - // Hash the token and do some element searching - theName = CHash::HashString(theToken); - - if (theName == RESERVED_PARENT) { - if (theElement) - theElement = theElement->GetParent(); - } else if (theName == RESERVED_THIS) { - ; - } else { - if (theName == RESERVED_SCENE && theParseCounter == 1) { - // theElement is nullptr, so using absolute path - theElement = thePresentation->GetRoot(); - } else if (theElement) { - // Using relative path - theElement = theElement->FindChild(theName); - } - } - - if (!theElement) - thePath = nullptr; - } // while - - Q3DStudio_free(theToken, CHAR, thePathLength); - return theElement; -} -} // namespace Q3DStudio diff --git a/src/runtime/Qt3DSElementHelper.h b/src/runtime/Qt3DSElementHelper.h deleted file mode 100644 index df2afdd..0000000 --- a/src/runtime/Qt3DSElementHelper.h +++ /dev/null @@ -1,57 +0,0 @@ -/**************************************************************************** -** -** Copyright (C) 1993-2009 NVIDIA Corporation. -** Copyright (C) 2017 The Qt Company Ltd. -** Contact: https://www.qt.io/licensing/ -** -** This file is part of Qt 3D Studio. -** -** $QT_BEGIN_LICENSE:GPL$ -** Commercial License Usage -** Licensees holding valid commercial Qt licenses may use this file in -** accordance with the commercial license agreement provided with the -** Software or, alternatively, in accordance with the terms contained in -** a written agreement between you and The Qt Company. For licensing terms -** and conditions see https://www.qt.io/terms-conditions. For further -** information use the contact form at https://www.qt.io/contact-us. -** -** GNU General Public License Usage -** Alternatively, this file may be used under the terms of the GNU -** General Public License version 3 or (at your option) any later version -** approved by the KDE Free Qt Foundation. The licenses are as published by -** the Free Software Foundation and appearing in the file LICENSE.GPL3 -** included in the packaging of this file. Please review the following -** information to ensure the GNU General Public License requirements will -** be met: https://www.gnu.org/licenses/gpl-3.0.html. -** -** $QT_END_LICENSE$ -** -****************************************************************************/ - -#pragma once - -namespace qt3ds { -namespace runtime { - class IApplication; -} -} - -namespace Q3DStudio { - -class IPresentation; - -class CElementHelper -{ -private: - CElementHelper(); - CElementHelper(const CElementHelper &); - CElementHelper &operator=(const CElementHelper &); - virtual ~CElementHelper(); - -public: - static TElement *GetElement(qt3ds::runtime::IApplication &inApplication, - IPresentation *inDefaultPresentation, const char *inPath, - TElement *inStartElement = nullptr); -}; - -} // namespace Q3DStudio |