[KRdp] [Bug 487543] KRDP KCM Page Takes a Few Seconds to Load and Locks Up System Settings.

[Akseli Lahtinen]

https://bugs.kde.org/show_bug.cgi?id=487543

Akseli Lahtinen <akselmo at akselmo.dev> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/krdp/-/commit/b0975b633a |ma/krdp/-/commit/4d65be1fb5
                   |44ffc61da429c4d4b94568bb0fd |57058fa3cff7e26d2f9cb377ef1
                   |437                         |879

--- Comment #5 from Akseli Lahtinen <akselmo at akselmo.dev> ---
Git commit 4d65be1fb557058fa3cff7e26d2f9cb377ef1879 by Akseli Lahtinen.
Committed on 28/05/2024 at 08:43.
Pushed by akselmo into branch 'Plasma/6.1'.

kcmkrdpserver: Use QDBusMessage instead of QDBusInterface

QDBusInterface blocks the GUI from updating. QDBusMessage do not.

Thus, use QDBusMessage, then QDBusConnection with asyncCall to talk to DBus,
hopefully avoiding any UI update blocking.

Possibly helps to fix


(cherry picked from commit b0975b633a44ffc61da429c4d4b94568bb0fd437)

4a8ee9c4 kcmkrdpserver: emit signal if server is running
552ec3e7 kcmkrdpserver: use QDBusMessage for toggling autoconnect
adf5cacd kcmkrdpserver: toggle server with qdbusmessage
5e2ec6a8 kcmkrdpserver: use qdbusmessage for checkserverrunning
8e594861 Use static variables for dbus names
918ff95f Delete watcher when finished
6302ee42 Use u""_qs instead of qstringliteral for readability

M  +32   -28   src/kcm/kcmkrdpserver.cpp
M  +2    -1    src/kcm/kcmkrdpserver.h
M  +6    -1    src/kcm/ui/main.qml

https://invent.kde.org/plasma/krdp/-/commit/4d65be1fb557058fa3cff7e26d2f9cb377ef1879

-- 
You are receiving this mail because:
You are the assignee for the bug.