[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