Code review comment for lp:~macslow/notify-osd/gravity

Revision history for this message
Mirco Müller (macslow) wrote :

> 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

« Back to merge proposal