> Thanks for putting that together quickly.
> 
> Remarks
> - line 57: the comment is wrong (mentions a font size)
> - it looks like the error rollback code could be factored into a helper
> function; but that's fine for a quick fix
> - unit tests, good

Fixed comment and added error-handling helper-function in rev. 389
-- 
https://code.edge.launchpad.net/~macslow/notify-osd/gravity/+merge/10963
Your team ayatana-commits is subscribed to branch lp:notify-osd.

_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp

Reply via email to