On Nov 26, 2007 2:24 PM, Julian Foad <julianfoad@btopenworld.com> wrote:
> David Glasser wrote:
> > On Nov 22, 2007 5:50 PM, David Glasser <glasser@davidglasser.net> wrote:
> >
> >>What's up with svn_repos_dir_delta[2] (libsvn_repos/delta.c)? It
> >>seems to me like it does a subset of the functionality of the reporter
> >>mechanism (libsvn_repos/reporter.c), with a completely duplicated
> >>implementation. It also seems like it isn't used much: just in
> >>svn_repos_dump_fs2, and in one bit of of the update report in
> >>mod_dav_svn. Is its implementation actually semantically different
> >>from the implementation of the reporter, or can it be reimplemented as
> >>a wrapper around the reporter? Is the separation just for historical
> >>reasons?
> >
> > [...] r8840 explains
> > it; the most obvious reason that svn_repos_dir_delta2 can't just be a
> > wrapper is that is works on txn roots, but the reporter doesn't.
>
> Reaching this level of awareness can be a big milestone in someone's
> understanding of the APIs. Now that you have discovered it, could you add in
> the respective API docs at least a hint that a relationship of similarity and
> significant differences exists between the two. If you can state some
> similarities and differences, so much the better.
>
> The next person along will be grateful.
Good call. r28057.
> >>(There are also many places in the code that refer to
> >>svn_repos_dir_delta[2] in comments, but which appear to be wrong
> >>because it's actually the reporter that implements what it is
> >>referring to...)
>
> Urgh.
Indeed. Forgive me for failing to fix this now, though.
--dave
--
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 27 02:59:36 2007