Re: svn commit: r19194 - in branches/fs-atomic-renames/subversion: include libsvn_delta libsvn_repos

On Thu, 06 Apr 2006, Peter N. Lundblad wrote:

> rooneg@tigris.org writes:
...
> > Log:
> > On the fs-atomic-renames branch:
> >
> > Rev the delta editor interface to support renames.
...
> Should we take the oportunity to call this svn_delta_editor_create to use
> a similar name to many other constructor functions?

+1, svn_delta_default_editor() is an aberration.

Also, should svn_delta_editor_to_editor2() really be in the public
header file? Seems like an implementation detail to me.

...
> When you do the bulk of this: change everything to use the new editor,
> will you move that work to trunk, then? I really hope you will.
> (Except for the rename stuff, of course).

This might complicate subsequent porting of the backend renames
changes, but if Garrett is up for that added pain, it would be nice to
have these API changes sooner than later.

-- 
Daniel Rall