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

Re: ephemeral txnprops

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Mon, 1 Jul 2013 21:01:33 +0300

C. Michael Pilato wrote on Mon, Jul 01, 2013 at 13:23:59 -0400:
> On 07/01/2013 11:54 AM, Daniel Shahaf wrote:
> > SVN_RA_CAPABILITY_EPHEMERAL_TXNPROPS seems to be queried only by the
> > libsvn_ra_serf and libsvn_ra_svn. Is there a reason not to either:
> >
> > - Make ra_local set those properties too
> > or
> > - Move the logic querying that capability (and when available setting
> > svn:txn-* revprops) to ra_loader.c
> >
> > ?
>
> There's no reason to query the capability in ra_local (tautology?). But
> yeah, I think it makes sense for ra_local to set the properties, too.

Done that in r1498608.

There's room for more code sharing (lift the get_client_string() call to
ra_loader.c), but I won't do it right now.

Cheers,

Daniel
Received on 2013-07-01 20:02:15 CEST

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