From 32b6f5e337d076530fc6fec638f192102217c292 Mon Sep 17 00:00:00 2001 From: Paul Lemire Date: Wed, 15 Apr 2020 10:23:02 +0200 Subject: 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 --- src/quick3d/imports/scene3d/scene3ditem.cpp | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src') 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 -- cgit v1.2.3