Re: svn commit: rev 3138 - trunk/subversion/libsvn_ra_dav

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

> Karl Fogel <kfogel@newton.ch.collab.net> writes:
>
> > Philip Martin <philip@codematters.co.uk> writes:
> > > rev 3113? Should svn_path_remove_component("/") return "/"?
> >
> > I think that would be the cleanest semantics, frankly. And if that
> > also happens to result in more code working right now, so much the
> > better :-)...
>
> It returns "/" now, and always did; that's why I am confused. What
> caused the infinite loop bug to appear, or was it always there?

It was always there, I just managed to trigger it. As
__get_baseline_info searches upwards through parent directories
retrying the request, some parent has always succeeded. But I found a
scenario where every parent failed, and thus revealed the infinite "/"
loop.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 6 04:10:32 2002