Re: CVS update: subversion/subversion/include svn_delta.h

Greg Hudson <ghudson@MIT.EDU> writes:
> So, here is a possibly better description of the issues I have:

Thanks for the summary. I'll try to address them:

> The current interface does not enforce that directory batons are used
> in depth-first order; that is only enforced through documentation. It
> would be easy to enforce depth-first order by doing away with
> dir_batons and storing the current directory stack inside the
> editor_baton.

Hmm, good point, I understand now. Give me an hour or so to bounce
ideas around with Ben and Jim verbally here, before forming an
opinion.

So do you think it would be better if we enforced depth-first order by
doing away with dir batons?

> The current interface does not enforce that apply_delta calls only
> come just after a file is opened or after the root baton is closed.
> This is harder to enforce through the interface, but it would be
> possible to come closer by eliminating file_batons and having the
> producer make an explicit call when a text delta is to be deferred.

Hm. Again, will digest for a bit and get back to you.
Received on Sat Oct 21 14:36:09 2006