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

Re: svn_wc_get_change_walker()

From: Karl Fogel <kfogel_at_galois.collab.net>
Date: 2000-09-02 04:53:29 CEST

I wrote:
> Greg, we should talk sometime about svn_wc_get_change_walker() and how
> it relates to svn_wc_apply_delta(). Just to sync up, I mean -- I see
> the rationale for the new function and all that, just want to make
> sure we don't end up duplicating code.
> In the meantime, I won't touch it, will just work on
> svn_wc_apply_delta().

Actually, now that I think about it more, I don't see why we'd have
both functions:

   - The only known purpose of svn_wc_apply_delta() is to effect
     updates/checkouts from some source (most likely the RA layer).

   - The only known purpose of svn_wc_get_change_walker() is to effect
     updates/checkouts from some source (most likely the RA layer).

Therefore, my first instinct is that we don't both of these. :-)

What is it that get_change_walker() gets us that apply_delta()
doesn't, or vice versa? Let's see if we can't merge these...

Received on Sat Oct 21 14:36:07 2006

This is an archived mail posted to the Subversion Dev mailing list.