----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125827/#review87511 -----------------------------------------------------------
ksmserver/screenlocker/ksldapp.cpp (lines 575 - 580) <https://git.reviewboard.kde.org/r/125827/#comment60089> as that's a string comparison each time, consider to add to ctor: bool m_isX11 = QX11Info::isPlatformX11(); bool m_isWayland = ... - Martin Gräßlin On Oct. 27, 2015, 12:18 p.m., Bhushan Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125827/ > ----------------------------------------------------------- > > (Updated Oct. 27, 2015, 12:18 p.m.) > > > Review request for Plasma and Martin Gräßlin. > > > Repository: plasma-workspace > > > Description > ------- > > At this moment WaylandLocker is just no-op, but X11Locker being highly X > dependant this provides default implementation for lockwindow. > > > Diffs > ----- > > ksmserver/screenlocker/CMakeLists.txt 367e81e > ksmserver/screenlocker/ksldapp.cpp f5dd998 > ksmserver/screenlocker/waylandlocker.h PRE-CREATION > ksmserver/screenlocker/waylandlocker.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/125827/diff/ > > > Testing > ------- > > works : http://i.imgur.com/d0ZCuO7.png > > > Thanks, > > Bhushan Shah > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
