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

Re: wc-ng code organisation

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Thu, 28 Oct 2010 17:26:13 +0100

Julian Foad <julian.foad_at_wandisco.com> writes:

> On Mon, 2010-10-25, Philip Martin wrote:
>> per tree).
>>
>> So the delete code ends up looking like
>>
>> svn_wc_delete4(path):
>> deleted = svn_wc__db_op_delete(path)
>> for name in deleted:
>> notify(name)
>> run workqueue
>> cleanup
>>
>> Does that sound sensible?
>>
>> Finally, while this mail refers to delete there are similar problems in
>> other functions: revert, copy, etc.
>
> +1 to the basic idea that the DB API is responsible for making a whole
> logical change, and the outside code shouldn't have to duplicate the
> logic.
>
> I've started to use a part of this pattern in the "copy" operation,

Once I started implementing it I realised that it didn't really solve
the particular delete problems I was facing. It might well be suitable
for copy.

The problem with delete is that the user can manually run delete on some
or all of the sub-trees, starting at the tip and working towards the
parent. Thus the final delete of the parent has to be able to cope with
all parts of the tree already being deleted.

-- 
Philip
Received on 2010-10-28 18:26:55 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.