Differential D1482

Connect Application::screensCreated to WaylandServer::initOutputs
ClosedPublic
Actions

Authored by graesslin on Apr 25 2016, 7:51 AM.

Details

Reviewers
sebas
Group Reviewers
Plasma
Commits
R108:319869881c87: Connect Application::screensCreated to WaylandServer::initOutputs
Summary

Instead of having the Application invoke initOutputs after creating
the Screens, we can just connect to the signal emitted there.

This allows to make initOutputs a private, WaylandServer internal
method.

Diff Detail

Repository
R108 KWin
Branch
init-outputs-on-connect
Lint
No Linters Available
Unit
No Unit Test Coverage
graesslin updated this revision to Diff 3501.Apr 25 2016, 7:51 AM
graesslin retitled this revision from to Connect Application::screensCreated to WaylandServer::initOutputs.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptApr 25 2016, 7:51 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
sebas accepted this revision.May 2 2016, 1:05 PM
sebas added a reviewer: sebas.
sebas added a subscriber: sebas.

Looks good.

This revision is now accepted and ready to land.May 2 2016, 1:05 PM
Closed by commit R108:319869881c87: Connect Application::screensCreated to WaylandServer::initOutputs (authored by graesslin). · Explain WhyMay 2 2016, 2:07 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 3501

View Options

autotests/wayland/kwin_wayland_test.cpp

Loading...
View Options

main_wayland.cpp

Loading...
View Options

wayland_server.h

Loading...
View Options

wayland_server.cpp

Loading...