[Differential] [Commented On] D3319: Load screenpool at the same time as we connect to screenchanged signals

fvogt (Fabian Vogt) noreply at phabricator.kde.org
Wed Nov 9 17:27:41 UTC 2016


fvogt added a comment.


  A quick question: This changes what the ScreenPool constructor does, are there any potential users of ScreenPool that now miss a call to ->load(), as it was done implicitly before? Wouldn't it be safer to call load() in the constructor as well?

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: fvogt, cfeck, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161109/6ae67ba7/attachment.html>


More information about the Plasma-devel mailing list