Review Request 123736: single QQmlEngine for all applets

Marco Martin notmart at gmail.com
Wed Jun 10 16:41:53 UTC 2015


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

(Updated June 10, 2015, 4:41 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Changes
-------

Submitted with commit 13bd6703c0dce6694ec0f06e8a3408cd3495ab29 by Marco Martin to branch master.


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/20150610/87264dc2/attachment.html>


More information about the Plasma-devel mailing list