Code review comment for lp:~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887

Revision history for this message
Robert Roth (evfool) wrote :

The fix looks fine, but there's a nonsense sentence left at the end of the description, although this was not reported in the bug ("as per the Desktop Notifications spec"). I guess it would also worth removing that at the same time, and the whole change should be forwarded to Debian (they should accept it without objections), to avoid having distro patches.

review: Needs Fixing

« Back to merge proposal