Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: 
 <CAEcU=1aRLG8PdyYNmm-8w5twehfxAZWosM699sJLHH3mhjTgiQ@mail.gmail.com>
References: <20150924135917.396E63A0234@svn01-us-west.apache.org>
 <002b01d0f6d6$24afce40$6e0f6ac0$@qqmail.nl>
 <CABw-3YcTARj0tvpZGswP4uosBH7ciuvN1DfzczZhrRhDJtGdtg@mail.gmail.com>
 <20150924155050.GE15939@jim.stsp.name>
 <CABw-3Ye8YO=_tMRFCco8D7ox3Pqv_h0XUV8ypuQrxdHn5GuQmA@mail.gmail.com>
 <CAEcU=1by8WrABSfkSLnco67mmjjirnmkFi7zVgjGEcq7VrC+5w@mail.gmail.com>
 <CABw-3YfjFnKvA+8NMspW8mqTt7WoDwmOX2k4ggztCdswfYhc7A@mail.gmail.com>
 <CAEcU=1aRLG8PdyYNmm-8w5twehfxAZWosM699sJLHH3mhjTgiQ@mail.gmail.com>
From: Ivan Zhakov <ivan@visualsvn.com>
Date: Thu, 24 Sep 2015 20:50:11 +0200
Message-ID: 
 <CABw-3Ye_=V-p+8Nd2MUrVkuOmyFfwd-p13mrM4wiEFFkJhgNXw@mail.gmail.com>
Subject: Re: svn commit: r1705060 - in
 /subversion/trunk/subversion/libsvn_ra_serf:
 ra_serf.h serf.c util.c
To: Julian Foad <julianfoad@btopenworld.com>
Cc: Bert Huijben <bert@qqmail.nl>, dev <dev@subversion.apache.org>
Content-Type: text/plain; charset=UTF-8

On 24 September 2015 at 19:15, Julian Foad <julianfoad@btopenworld.com> wrote:
> Ivan Zhakov wrote:
>> Julian Foad wrote:
>>> Ivan Zhakov wrote:
>>>> [...] we also have many functions that accepts just POOL and use
>>>> it as scratch pool. And we also have many functions that uses it as
>>>> result pool.
>>>
>>> Yes, we do have many of those. That was the Old Way. Naming the pools
>>> 'scratch_pool' or 'result_pool' is the New Way. We seem to generally
>>> agree that is better, and sometimes we rename the single 'pool'
>>> argument of old functions to either 'scratch_pool' or 'result_pool'.
>>
>> Could you please give me link to the thread where we discussed The New
>> Way? Yes, we use result_pool/scratch_pool, but I don't remember
>> discussion about never using just one POOL.
>
> I don't know if this was ever explicitly discussed. (The thread where
> the two-pools paradigm was first publicly is: Subject "result_pool and
> scratch_pool", from Greg Stein, on 2008-10-06, archived at e.g.
> http://svn.haxx.se/dev/archive-2008-10/0268.shtml )
>
As far I remember this thread about  two pools paradigm itself
(result_pool and scratch_pool). It's not about making all functions
follow two pools paradigm.  Some of API introduced in 1.9 doesn't
follow this paradigm for some reason. Please understand me correctly:
I really like two pool paradigm, but sometimes it's not necessary,
especially for local helpers.

-- 
Ivan Zhakov