diff options
author | Ulf Hermann <ulf.hermann@qt.io> | 2021-01-13 12:33:06 +0100 |
---|---|---|
committer | Ulf Hermann <ulf.hermann@qt.io> | 2021-01-13 14:32:34 +0100 |
commit | cfe0b08b5439a27b4fdd14c29620e0492543f506 (patch) | |
tree | e45d282b23bec6504dbaba2b784bbaaa44f0b16b /src/quick/designer | |
parent | b64f8dacae36fca948933cf56498d5e4ad3e2a07 (diff) |
Remove QQmlCleanup
The only place where it was still used is QQmlOpenMetaObjecType. The
only effect of QQmlCleanup is that clear() is eventually called by the
engine. In the case of QQmlOpenMetaObjectType the only effect of that
was that the "engine" member was reset. However, the only place where
that member was used was in setCached(), and in that place it was
irrelevant. There is no reason why setCached() should be prohibited when
there is no engine. We may be worried that the property cache assigned
to the open metaobject might go away somehow if there is no engine, but
the cleanup mechanism clearly demonstrates that checking for the engine
on setCached() does not protect against this.
Fixes: QTBUG-90004
Change-Id: I05445eaeb53a64c70de366090ea2ee4aecf2bad8
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/quick/designer')
-rw-r--r-- | src/quick/designer/qqmldesignermetaobject.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/quick/designer/qqmldesignermetaobject.cpp b/src/quick/designer/qqmldesignermetaobject.cpp index 0aa5dc2613..0264b751f1 100644 --- a/src/quick/designer/qqmldesignermetaobject.cpp +++ b/src/quick/designer/qqmldesignermetaobject.cpp @@ -110,7 +110,7 @@ QQmlDesignerMetaObject* QQmlDesignerMetaObject::getNodeInstanceMetaObject(QObjec void QQmlDesignerMetaObject::init(QObject *object, QQmlEngine *engine) { //Creating QQmlOpenMetaObjectType - m_type = new QQmlOpenMetaObjectType(metaObjectParent(), engine); + m_type = new QQmlOpenMetaObjectType(metaObjectParent()); m_type->addref(); //Assigning type to this copyTypeMetaObject(); |