Re: [PATCH] issue #2147 - v2 - control characters in log message cause failure (over DAV)

VK Sameer wrote:
> The test has been modified to run in ra_dav mode only. If there is
> a better way to do it than by checking svntest.main.test_area_url,
> please let me know.

I don't know.

> +V 59
> +this msg contains a Ctrl-T><, a Ctrl-I> <, and a Ctrl-M>
> +<

The version I received doesn't contain a Ctrl-M. :-)

We definitely need a better way to put these characters into the test string.

   http://docs.python.org/ref/strings.html

So you could write "a Ctrl-T>\x14<, a Ctrl-I>\t<, and a Ctrl-M>\r<".

All other comments on this patch were in my reply to the version 1 discussion.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jan 28 18:05:07 2005