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

Re: svn commit: r38202 - trunk/subversion/libsvn_wc

From: Hyrum K. Wright <hyrum_at_hyrumwright.org>
Date: Fri, 26 Jun 2009 11:31:13 -0500

On Jun 26, 2009, at 12:37 AM, Greg Stein wrote:

> On Fri, Jun 26, 2009 at 06:19, Hyrum K.
> Wright<hyrum_wright_at_mail.utexas.edu> wrote:
>> ...
>>> Looking at the code, I think what you really want is to examine
>>> parent_relpath, NOT do a LIKE clause.
>>
>> The real problem here is getting the transitive closure on all the
>> nodes
>> which have a given parent. So while using parent_relpath would
>> find all the
>> immediate children, we'd need to continue to recurse to nail every
>> potential
>> child node. That is, unless not grasping an important bit of our
>> repository
>> information inheritance scheme.
>
> Ah. Hrm. Right... This isn't the "relocate the children" thing (in
> another rev), but "relocate the tree".
>
> Sigh.
>
> Okee dokey, then. Seems fine, except that you used apr_pcalloc() when
> a palloc would be just fine. You're filling every byte manually, so
> there is no reason to zero it beforehand.

I'm open to other suggestions to get around the LIKE clause, but
figured we'd hit those up in the post-implementation optimization pass.

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2365768
Received on 2009-06-26 18:31:38 CEST

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.