Re: svn commit: rev 6318 - trunk/subversion/libsvn_delta

Philip Martin <philip@codematters.co.uk> writes:

> I don't really know the ra_dav editor code that well, but looking at
> it now see how commit_open_dir in libsvn_ra_dav/commit.c only uses the
> last component of the input path, the rest comes from the "known"
> parent. This meant that the ra_dav editor would construct an invalid
> path. Perhaps this function should do a sanity check on the whole
> path?

That seems sensible.

> Or perhaps the editor interface should change and only pass the last
> component?

Nah. That how things used to be, and we really don't want to return
to that place.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 23 06:17:43 2003