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

Re: svn commit: r1657451 - in /subversion/branches/reuse-ra-session: ./ subversion/libsvn_client/

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Thu, 5 Feb 2015 17:30:25 +0300

On 5 February 2015 at 17:22, Branko Čibej <brane_at_wandisco.com> wrote:
> On 05.02.2015 11:13, Ivan Zhakov wrote:
>> On 5 February 2015 at 03:44, <brane_at_apache.org> wrote:
>>> Author: brane
>>> Date: Thu Feb 5 00:44:57 2015
>>> New Revision: 1657451
>>>
>>> URL: http://svn.apache.org/r1657451
>>> Log:
>>> On the reuse-ra-session branch: Add explicit session reuse in libsvn_client.
>>>
>> Cool stuff! Two minor nits are below.
>
> Thanks! Fixed in r1657574, along with a few more of the same ilk.
>
> I prefer to keep the svn_error_t* return from the release function and
> write code as if it actually can fail,.
>
I don't have opinion what is better, so I'm fine with both approaches.
Anyway this is private API and we can change it anytime.

-- 
Ivan Zhakov
Received on 2015-02-05 15:32:41 CET

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.