summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAntti Määttä <antti.maatta@qt.io>2020-01-20 08:38:32 +0200
committerAntti Määttä <antti.maatta@qt.io>2020-01-20 11:31:05 +0200
commit64f4a0f31565dce726d1306c0e36dea517bf80f3 (patch)
treebe4e802d5a374571c9b4825de7e40a564f201f4e
parent29eab4197f6cd2d6725c71bc612356c43f872993 (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.cpp4
-rw-r--r--src/ogl-runtime-static/ogl-runtime-static.pro2
-rw-r--r--src/runtime/Qt3DSActivationManager.cpp15
-rw-r--r--src/runtime/Qt3DSElementHelper.cpp141
-rw-r--r--src/runtime/Qt3DSElementHelper.h57
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