Re: svn_wc_get_change_walker()

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...

-Karl
Received on Sat Oct 21 14:36:07 2006