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

Re: [PATCH] Re: slow status

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2004-10-10 18:14:39 CEST

On Sun, 2004-10-10 at 12:04, Greg Hudson wrote:
> I think doing that can also simplify the logic. If there is a target,
> lock the anchor non-recursively and then lock the target (recursively or
> not, as specified). If there is no target, lock the anchor recursively
> or not as specified. That's fewer cases than your patch has.

That was imprecise and missed the subtlety that we need to lock the
subdirs for a non-recursive status. But there's still no need to
consider more cases than I specified. So, to be more precise:

  If there is a separate anchor and target:
    Lock the anchor with depth 0
    Lock the target with depth (recursive ? -1 : 1)
    Lock the anchor with depth (recursive ? -1 : 1)

No need to consider whether the target is a file or anything like that.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Oct 10 18:14:51 2004

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.