[PATCH 1 of 1] tests: asciify output of tsttest

Yuya Nishihara yuya at tcha.org
Sun Sep 19 08:50:14 CDT 2010


Martin Geisler wrote:
> Yuya Nishihara <yuya at tcha.org> writes:
> > +def asciify(line):
> > +    """Escape non-ascii and most control characters"""
> > +    return re.sub(r'[^\x20-\x7f\n\t]',
> > +                  lambda m: m.group(0).encode('string-escape'), line)
> 
> I like this change a lot, but I don't see why you only encode that range
> of characters?

To avoid escape of ' and \.
It sounds noisy for me.

Yuya,


More information about the Mercurial-devel mailing list