Review Request: small FIXME code cleanup in ui/notificationgroup.*

Martin Koller kollix at aon.at
Mon Aug 6 21:59:33 UTC 2012


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

Review request for Plasma and Marco Martin.


Description
-------

The header contained a "FIXME" line which this patch solves.
Also, an unneeded member variable was removed as it was only used locally


Diffs
-----

  plasma/generic/applets/notifications/ui/notificationgroup.h 686442f 
  plasma/generic/applets/notifications/ui/notificationgroup.cpp 7ea9355 

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


Testing
-------


Thanks,

Martin Koller

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120806/85fdb754/attachment-0001.html>


More information about the Plasma-devel mailing list