Code review comment for lp:~bac/launchpad/bug-720147

Revision history for this message
Benji York (benji) wrote :

This branch looks good.

The only thought I had was that the part starting on line 13 of the diff
is a bit repetitive and I had to look very closely to see if it was
exactly the same pattern for each line or if one or more were subtly
different. However, I'm not sure my version is any better:
http://pastebin.ubuntu.com/602393/ (untested).

review: Approve (code)

« Back to merge proposal