D22225: Partly remove Compositor restart functionality

Roman Gilg noreply at phabricator.kde.org
Tue Jul 2 21:42:11 BST 2019


romangg created this revision.
romangg added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
romangg requested review of this revision.

REVISION SUMMARY
  This removes the dbus method to reinit the compositor and the restart
  of the Compositor class.
  
  The dbus method seems to be of little use and also did not work when
  tried last on master.
  
  Instead of the restart function reinitialize can be called. Reading again
  the settings in this case is fine, since it is done rarely. This reduces
  the code complexity.

TEST PLAN
  Manually on Wayland and X. 100% autotests pass.

REPOSITORY
  R108 KWin

BRANCH
  rmReinitRestart

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

AFFECTED FILES
  autotests/integration/dont_crash_reinitialize_compositor.cpp
  autotests/integration/generic_scene_opengl_test.cpp
  autotests/integration/scene_qpainter_test.cpp
  composite.cpp
  composite.h
  dbusinterface.cpp

To: romangg, #kwin
Cc: kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190702/ddeef5a8/attachment-0001.html>


More information about the kwin mailing list