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