Re: [PATCH] clean up some debugging code

On Thu, May 08, 2008 at 10:19:34AM +0200, Stefan Sperling wrote:
> Hi,
>
> comments, anyone?
>
> I'd like to commit this, but have one open question (read the log
> message first, my question is below).
>
> The patch passes 'make check'.

> @@ -347,7 +367,7 @@ svn_path_add_component(svn_stringbuf_t *path,
> void
> svn_path_remove_component(svn_stringbuf_t *path)
> {
> - assert(is_canonical(path->data, path->len));
> + assert(is_canonical(path->data, strlen(path->data)));

Mmmmh. There's a few redundant strlen calls in it now alright,
they happened because I did a global search/replace edit.

I will fix those, i.e. use ->len again where possible, before commit.

Stefan