[Differential] [Request, 21 lines] D2605: [Desktop Containment] Create AppletAppearance component only once
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Sat Aug 27 17:55:55 UTC 2016
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Instead cache the created component. Also check its status rather than doing a string comparison on errorString and only call it if it actually failed.
Moreover, initialize AppletAppearnce already with category set.
TEST PLAN
Still works.
QML Engine has a component cache but still I think calling Qt.createComponent repeatedly (especially since gc will collect it later on when it falls out of scope) should be avoided?
REPOSITORY
rPLASMADESKTOP Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D2605
AFFECTED FILES
containments/desktop/package/contents/ui/main.qml
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160827/82f16807/attachment-0001.html>
More information about the Plasma-devel
mailing list