Rename Client to X11Client
ClosedPublic

Authored by zzag on Sep 24 2019, 10:02 AM.

Details

Summary

Currently each managed X11 client is represented with an instance of
Client class, however the name of that class is very generic and the
only reason why it's called that way is because historically kwin
was created as an x11 window manager, so "Client" was a sensible choice.

With introduction of wayland support, things had changed and therefore
Client needs to be renamed to X11Client in order to better reflect what
that class stands for.

Renaming of Client to X11Client was agreed upon during the last KWin
sprint.

Test Plan

Compiles, the test suite is still green.

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.
zzag created this revision.Sep 24 2019, 10:02 AM
Restricted Application added a project: KWin. · View Herald TranscriptSep 24 2019, 10:02 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Sep 24 2019, 10:02 AM
zzag edited the summary of this revision. (Show Details)Sep 24 2019, 10:03 AM

I'm fine with this renaming. @davidedmundson; anything to add before this lands?

romangg accepted this revision.Sep 25 2019, 4:52 PM
This revision is now accepted and ready to land.Sep 25 2019, 4:52 PM
This revision was automatically updated to reflect the committed changes.