Code review comment for lp:~gary/launchpad/bug_475371

Revision history for this message
Gary Poster (gary) wrote :

Thanks, Leonard. I see the mistake I made: I was not including the DOCTYPE. I did len(c.split('\r\n',5)[-1]) instead of len(c.split('\r\n',4)[-1]). But now I get 1188, not 1189. I think I will follow your advice and remove it, and see if the LOSAs say that will work.

« Back to merge proposal