[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1466659 - /subversion/trunk/subversion/svnserve/serve.c

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Thu, 11 Apr 2013 00:31:02 +0400

On Thu, Apr 11, 2013 at 12:28 AM, <julianfoad_at_apache.org> wrote:
> Author: julianfoad
> Date: Wed Apr 10 20:28:42 2013
> New Revision: 1466659
>
> URL: http://svn.apache.org/r1466659
> Log:
> Fix issue #4348, "missing directory replace on update 1.8 svnserve 1.7 client".
>
> * subversion/svnserve/serve.c
> (update): Correct the backward-compatibility code for the
> 'ignore_ancestry' parameter introduced in r1465292.
>
> Found by: philip
>
> Modified:
> subversion/trunk/subversion/svnserve/serve.c
>
> Modified: subversion/trunk/subversion/svnserve/serve.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svnserve/serve.c?rev=1466659&r1=1466658&r2=1466659&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/svnserve/serve.c (original)
> +++ subversion/trunk/subversion/svnserve/serve.c Wed Apr 10 20:28:42 2013
> @@ -1814,7 +1814,7 @@ static svn_error_t *update(svn_ra_svn_co
> const char *target, *full_path, *depth_word;
> svn_boolean_t recurse;
> apr_uint64_t send_copyfrom_args; /* Optional; default FALSE */
> - apr_uint64_t ignore_ancestry; /* Optional; default TRUE */
> + apr_uint64_t ignore_ancestry; /* Optional; default FALSE */
> /* Default to unknown. Old clients won't send depth, but we'll
> handle that by converting recurse if necessary. */
> svn_depth_t depth = svn_depth_unknown;
> @@ -1842,7 +1842,7 @@ static svn_error_t *update(svn_ra_svn_co
> conn, pool, b, rev, target, NULL, TRUE,
> depth,
> (send_copyfrom_args == TRUE) /* send_copyfrom_args */,
> - (ignore_ancestry != FALSE) /* ignore_ancestry */));
> + (ignore_ancestry == TRUE) /* ignore_ancestry */));
As far I remember any non-zero value considered as boolean 'true'. So
correct expression should be just 'ignore_ancestry'.

-- 
Ivan Zhakov
Received on 2013-04-10 22:31:53 CEST

This is an archived mail posted to the Subversion Dev mailing list.