On Thu, May 08, 2008 at 10:26:24AM +0200, Stefan Sperling wrote:
> 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.
No comments means consent, right? :)
(Well, actually Daniel said the patch was OK in IRC.)
Committed in r31121, forgot to fix it to use ->len though, that's
done in r31122.
The default for --enable-debug is still an open question, but can be
addressed separately.
Stefan
- application/pgp-signature attachment: stored
Received on 2008-05-11 11:54:00 CEST