Add Wayland support
ClosedPublic

Authored by stikonas on Sun, Feb 10, 7:40 PM.

Details

Test Plan

krdc can now connect on Wayland session.

The remote window is not embedded in krdc yet but still better than not connecting at all.

Diff Detail

Repository
R436 KRDC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
stikonas requested review of this revision.Sun, Feb 10, 7:40 PM
stikonas created this revision.
stikonas retitled this revision from Add wayland support to Add Wayland support.

Coding style: it looks like most places in this file uses {} also around one line ifs.

rdp/rdpview.cpp
366–371

Isn't there something in KWindowSystem that does this prettier ? I think I have reviewed something like this in the past there.

367

Are there proper error handling fi the executables aren't found?

stikonas updated this revision to Diff 51355.Sun, Feb 10, 9:07 PM
stikonas marked 2 inline comments as done.

Added more error handling if executable is not found.

Use KWindowSystem to detect Wayland

stikonas updated this revision to Diff 51369.Sun, Feb 10, 10:20 PM

Coding style adjusted

This revision was not accepted when it landed; it landed in state Needs Review.Tue, Feb 12, 11:29 PM
This revision was automatically updated to reflect the committed changes.