[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1496886 - /subversion/trunk/subversion/libsvn_client/ra.c

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Wed, 26 Jun 2013 17:50:58 +0200

On Wed, Jun 26, 2013 at 2:52 PM, Stefan Sperling <stsp_at_elego.de> wrote:

> On Wed, Jun 26, 2013 at 02:46:29PM +0200, Stefan Sperling wrote:
> > I think this block of code should either enforce and fulfill the
> > API contract in the exact same way as svn_ra_get_location_segments()
> > does, or it should not exist at all. I don't think we should bother
> > with optimizations that don't behave exactly like the non-optimized case.
>
> Another idea: what about moving this special case handling into
> libsvn_ra itself, rather than having it in libsvn_client?
>

The server-side implementation is pretty efficient already.
This has been about the extra network roundtrip (Interwebs!).

r1496957 accomplishes that optimization now.

-- Stefan^2.
Received on 2013-06-26 17:51:30 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.