diff options
author | Paul Lemire <paul.lemire@kdab.com> | 2020-04-15 10:23:02 +0200 |
---|---|---|
committer | Paul Lemire <paul.lemire@kdab.com> | 2020-04-15 10:28:44 +0200 |
commit | 32b6f5e337d076530fc6fec638f192102217c292 (patch) | |
tree | 367e04374503f19a820e87a192bc16ff5fabac84 | |
parent | 1efa5aa48b2922fa57d2b2d645b8665b9a71c1e9 (diff) | |
download | qt3d-32b6f5e337d076530fc6fec638f192102217c292.tar.gz |
Fix scene3ditem to register optional aspects
applyAspect shouldn't not be checking against the m_aspects list
following the recent refactorings introduced in
cdc9efb609894298d8d854a0ec75f9cb8e89f195.
This checks prevents any optional aspect from being used because
m_aspects will never be empty.
Change-Id: I6548d86599b3ac4c8aaad034a3e4ffc830a855c5
Reviewed-by: Mike Krus <mike.krus@kdab.com>
-rw-r--r-- | src/quick3d/imports/scene3d/scene3ditem.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/quick3d/imports/scene3d/scene3ditem.cpp b/src/quick3d/imports/scene3d/scene3ditem.cpp index 1444eb0a0..8717e320e 100644 --- a/src/quick3d/imports/scene3d/scene3ditem.cpp +++ b/src/quick3d/imports/scene3d/scene3ditem.cpp @@ -169,10 +169,6 @@ Qt3DCore::QEntity *Scene3DItem::entity() const void Scene3DItem::applyAspects() { - if (!m_aspects.isEmpty()) { - qWarning() << "Aspects already set on the Scene3D, ignoring"; - return; - } // Aspects are owned by the aspect engine for (const QString &aspect : qAsConst(m_aspects)) { if (aspect == QLatin1String("render")) // This one is hardwired anyway |