D15187: Merge switch user dialog into lockscreen

David Edmundson noreply at phabricator.kde.org
Sat Sep 1 08:29:23 BST 2018


davidedmundson added inline comments.

INLINE COMMENTS

> mart wrote in server.cpp:1095
> should we need some confirmation from the screenlocker that the operation actually went well?

good question.

The method is void, but we could still throw an error.

I checked all our code that calls this and they don't check the return.
If have to to change the calling code, I'll be changing it to call the method on the screensaver directly anyway. 
Proxying here is just for compatibility.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D15187

To: davidedmundson, #plasma, mart
Cc: ngraham, mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180901/aae353a3/attachment-0001.html>


More information about the Plasma-devel mailing list