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

Re: [PATCH]: Doc clarification for svn_wc__acquire_write_lock()

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Thu, 11 Nov 2010 16:52:33 +0000

On Tue, 2010-11-09 at 17:38 +0000, Julian Foad wrote:
> Can someone check this doc clarification for me?

For the record, I committed this in r1033861.

- Julian

> The condition that @a local_abspath must be a versioned directory if @a
> lock_anchor is false was hidden away in a sub-condition. This promotes
> it to an unconditional condition. :-)
>
> [[[
> Index: subversion/include/private/svn_wc_private.h
> ===================================================================
> --- subversion/include/private/svn_wc_private.h (revision 1033064)
> +++ subversion/include/private/svn_wc_private.h (working copy)
> @@ -614,18 +614,18 @@
> /**
> - * Acquire a recursive write lock for @a local_abspath or if @a lock_anchor
> + * Acquire a recursive write lock for @a local_abspath. If @a lock_anchor
> * is true, determine if @a local_abspath has an anchor that should be locked
> - * instead. Store the obtained lock in @a wc_ctx.
> + * instead; otherwise, @a local_abspath must be a versioned directory.
> + * Store the obtained lock in @a wc_ctx.
> *
> * If @a lock_root_abspath is not NULL, store the root of the lock in
> * @a *lock_root_abspath. If @a lock_root_abspath is NULL, then @a
> - * local_abspath must be a versioned directory and @a lock_anchor must be
> - * FALSE.
> + * @a lock_anchor must be FALSE.
> *
> * Returns @c SVN_ERR_WC_LOCKED if an existing lock is encountered, in
> * which case any locks acquired will have been released.
> *
> * If @a lock_anchor is TRUE and @a lock_root_abspath is not NULL, @a
> * lock_root_abspath will be set even when SVN_ERR_WC_LOCKED is returned.
> */
> svn_error_t *
> svn_wc__acquire_write_lock(const char **lock_root_abspath,
> ]]]
Received on 2010-11-11 17:53:14 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.