D16425: Added new Suspend then Hibernate option

Nathaniel Graham noreply at phabricator.kde.org
Fri May 17 17:34:46 BST 2019


ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  We just branched 5.16, so this will be 5.17 material. That should leave lots of time for testing. Would also be nice to get a review from @broulik once he returns from vacation or anyone else in #plasma <https://phabricator.kde.org/tag/plasma/>.
  
  Can you land the patch yourself or do you need someone else to do it for you?

INLINE COMMENTS

> powerdevilpowermanagement.cpp:207
> +{
> +    if(!d->serviceRegistered) {
> +        return;

space after `if`

> powerdevilpowermanagement.cpp:210
> +    }
> +    if(!d->canSuspendThenHibernate) {
> +        return;

space after `if`

REPOSITORY
  R122 Powerdevil

BRANCH
  arcpatch-D16425_1

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

To: avaldes, broulik, ngraham
Cc: reverendhomer, meven, soriano, abalaji, graesslin, ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190517/013c6953/attachment.html>


More information about the Plasma-devel mailing list