diff options
| author | Sean Harmer <sean.harmer@kdab.com> | 2016-04-10 09:05:27 +0100 |
|---|---|---|
| committer | Paul Lemire <paul.lemire@kdab.com> | 2016-04-12 12:44:57 +0000 |
| commit | e8a263f615ed18669b3ed85aa25a36b34ee1508a (patch) | |
| tree | 58e944bd7374ce3ba81123c22a2d6cdc944f38b9 /src/core/aspects/qabstractaspect.cpp | |
| parent | 6737083f5c743820af5e7c0424f265911a412ba7 (diff) | |
Query cloning vs no cloning once (per aspect)
This will allow to try out the no cloning approach on hotter code paths
without readign the environment variable every time.
Change-Id: I5cd4532ef942496640b09984784eb80e2203ebce
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Diffstat (limited to 'src/core/aspects/qabstractaspect.cpp')
| -rw-r--r-- | src/core/aspects/qabstractaspect.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/aspects/qabstractaspect.cpp b/src/core/aspects/qabstractaspect.cpp index 51658b470..5b27fe4b0 100644 --- a/src/core/aspects/qabstractaspect.cpp +++ b/src/core/aspects/qabstractaspect.cpp @@ -59,6 +59,7 @@ QAbstractAspectPrivate::QAbstractAspectPrivate() , m_aspectManager(Q_NULLPTR) , m_jobManager(Q_NULLPTR) , m_arbiter(Q_NULLPTR) + , m_useCloning(!qEnvironmentVariableIsSet("QT3D_NO_CLONE")) { } @@ -252,8 +253,7 @@ void QAbstractAspectPrivate::setRootAndCreateNodes(QEntity *rootObject, const QV m_rootId = rootObject->id(); // Use old method for now, unless user explicitly requests new method - const auto useNewMethod = qEnvironmentVariableIsSet("QT3D_NO_CLONE"); - if (!useNewMethod) { + if (m_useCloning) { QNodeVisitor visitor; visitor.traverse(rootObject, this, &QAbstractAspectPrivate::createBackendNode); } else { |