D8436: use a single QML engine

Marco Martin noreply at phabricator.kde.org
Mon Oct 23 15:14:40 UTC 2017


mart created this revision.
mart added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  use a shared qml engine for all ConfigModules in the same process
  this avoids some crashes as creating and deleting stuff from
  different engines is known to cause crashes in the v4 garbage collection
  and is recomended to use a single QQmlEngine per process

TEST PLAN
  normal SystemSettings seems unaffected, plasma mobile systemsettings
  which is qml-only now doesn't crash anymore after loading a module
  for the second time

REPOSITORY
  R296 KDeclarative

BRANCH
  phab/singleengine

REVISION DETAIL
  https://phabricator.kde.org/D8436

AFFECTED FILES
  src/quickaddons/configmodule.cpp

To: mart, #plasma
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171023/0de00c56/attachment.html>


More information about the Plasma-devel mailing list