[Kde-hardware-devel] Review Request: Tidy up PowerDevil notifyrc

Kai Uwe Broulik kde at privat.broulik.de
Tue Oct 2 13:43:40 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106689/
-----------------------------------------------------------

Review request for Solid.


Description
-------

That should be the last one ;)

This removes the notifications:
 - inhibition. (Suspend job has been inhibited because an app requested it)
   I could not find any cpp file in kde-workspace, kde-baseapps and kdelibs that uses this notification
 - lockscreen/suspendjob (The screen is being locked)
   I forgot to commit this with the last review. This has already been agreed on.
 - joberror (There was an error while performing a job)
   Could not find anyone that used that

Correct me if I overlooked something.
   


Diffs
-----

  powerdevil/daemon/actions/bundled/suspendsession.cpp 8369413 
  powerdevil/powerdevil.notifyrc 0e9ac81 

Diff: http://git.reviewboard.kde.org/r/106689/diff/


Testing
-------

Compiles and notification settings are much more tidy :)


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20121002/c721ccc7/attachment.html>


More information about the Kde-hardware-devel mailing list