[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