Review Request 123736: single QQmlEngine for all applets

Marco Martin notmart at gmail.com
Tue May 12 15:30:22 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123736/
-----------------------------------------------------------

Review request for KDE Frameworks and Plasma.


Repository: plasma-framework


Description
-------

use a single shared QQmlEngine for all applets, tooltip and wallpapers (each of which used to use an own one)
views are still separed engines because being View subclass of QQuickView which api unfortunately requires it (a qquickengine can be passed to the ctor, but in that case they would share their QQmlContext, which wouldn't work)

For compatibility, applets without X-Plasma-RequiredExtensions=LaunchApp will still have own engine


Diffs
-----

  src/declarativeimports/core/tooltipdialog.cpp feeacc3 
  src/plasmaquick/appletquickitem.cpp c6fcf22 
  src/plasmaquick/packageurlinterceptor.h 36c85a9 
  src/plasmaquick/packageurlinterceptor.cpp 5f77c3a 
  src/plasmaquick/private/appletquickitem_p.h a1ec683 
  src/scriptengines/qml/plasmoid/wallpaperinterface.cpp fcaca4b 

Diff: https://git.reviewboard.kde.org/r/123736/diff/


Testing
-------


Thanks,

Marco Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150512/944dc410/attachment.html>


More information about the Plasma-devel mailing list