On Wed, 2010-11-10, Noorul Islam K M wrote:
> [[[
> 
> Canonicalize dirent/URL before passing to client API

Thanks, Noorul.  I haven't checked this out yet, but it looks good.

In several places you write:

> SVN_ERR(svn_cl__revprop_prepare(&opt_state->start_revision, targets,
>                                        &URL, ctx, pool));
> +      URL = svn_uri_canonicalize(URL, pool);

It seems to me that that canonicalization would be better in
svn_cl__revprop_prepare().  Do you agree?

I'll check and commit this patch later.  The above change can be a
follow-up.

- Julian



Reply via email to