You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Blair Zajac <bl...@orcaware.com> on 2010/06/28 16:14:05 UTC

Re: svn commit: r958559 - in /subversion/trunk/subversion: include/ libsvn_client/ libsvn_delta/ libsvn_subr/ libsvn_wc/ mod_dav_svn/ svn/ tests/libsvn_subr/

On 06/28/2010 05:45 AM, rhuijben@apache.org wrote:
> Author: rhuijben
> Date: Mon Jun 28 12:45:39 2010
> New Revision: 958559
>
> URL: http://svn.apache.org/viewvc?rev=958559&view=rev
> Log:
> Switch the argument order of (the new in 1.7) svn_dirent_split(),
> svn_uri_split() and svn_relpath_split() functions to our new standard
> order: output arguments before input arguments.


> Modified: subversion/trunk/subversion/tests/libsvn_subr/dirent_uri-test.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_subr/dirent_uri-test.c?rev=958559&r1=958558&r2=958559&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/tests/libsvn_subr/dirent_uri-test.c (original)
> +++ subversion/trunk/subversion/tests/libsvn_subr/dirent_uri-test.c Mon Jun 28 12:45:39 2010
...
> @@ -1380,7 +1380,7 @@ test_relpath_split(apr_pool_t *pool)
>       {
>         const char *dir, *base_name;
>
> -      svn_relpath_split(paths[i][0],&dir,&base_name, pool);
> +      svn_relpath_split( &dir,&base_name, paths[i][0], pool);

A whitespace snuck in here.

Regards,
Blair