Re: CVS update: subversion/subversion/libsvn_fs dag.c editor.c

Branko =?ISO-8859-2?Q?=C8ibej?= <brane@xbc.nu> writes:
> I think we'll have to know the node's path so that we can create "copy"
> nodes from dag_node_t's sources. We have that information in
> svn_fs_node_t; we can
>
> a) move it to dag_node_t instead, or
> b) pass it to svn_fs__dag_make_copy explicitly (and hope to get it right).
>
> I'd prefer a), but it would cause a few changes in tree.c.

No, that's not right. The creation of new revisions of existing nodes
is something that needs to be explicitly exposed in dag.c's public
interface. The whole point of dag.c is to give us a boundary within
which we just worry about keeping the dag structure consistent, and
leave the hairy parent cloning concerns to the next level up.
Received on Sat Oct 21 14:36:21 2006