Code review comment for lp://staging/~lifeless/bzr/bug-3918

Revision history for this message
Andrew Bennetts (spiv) wrote :

Looks sane.

(It would be nice to actually support \r and \n in filenames, but I see that even 2a still barfs on that :( )

review: Approve

« Back to merge proposal