Remove KWin::display from kwinglobals
ClosedPublic

Authored by graesslin on Nov 11 2016, 9:05 AM.

Details

Summary

And finally nothing inside libkwineffects, libkwinglutils,
libkwinxrenderutils and kwineffect and kwin core uses KWin::display.
We are finally XLib free!

This change drops KWin::display and removes the include to QX11Info from
kwinglobals.h. And the libraries no longer need to link X11Extras. Due
to that removal a few seeming unrelated changes are required to add the
include where needed and linkage to X11Extras.

The biggest change is to x11 platform plugin which still needs the
display and caches it in the Platform and passes it to various places in
a way that the code doesn't need to be adjusted.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 8086.Nov 11 2016, 9:05 AM
graesslin retitled this revision from to Remove KWin::display from kwinglobals.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added projects: Plasma on Wayland, KWin. · View Herald TranscriptNov 11 2016, 9:05 AM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
mart accepted this revision.Nov 15 2016, 11:01 AM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Nov 15 2016, 11:01 AM
This revision was automatically updated to reflect the committed changes.