On 26 August 2016 at 13:36, Stefan Sperling <stsp_at_elego.de> wrote:
> On Fri, Aug 26, 2016 at 01:15:16PM +0300, Ivan Zhakov wrote:
>> Hi Stefan,
>>
>> Currently svn_client_conflict_t stores a pointer to svn_client_ctx_t
>> inside and uses it for every operation like
>> svn_client_conflict_tree_get_details() or
>> svn_client_conflict_*_resolve(). It may be useful for simple cases,
>> but for other cases it makes using new API more difficult. That's
>> because an API user needs to guarantee that lifetime of
>> svn_client_ctx_t is longer than of the svn_client_conflict_t instance.
>>
>> I suggest changing svn_client_conflict_* API so that it wouldn't keep
>> reference to svn_client_ctx_t and add it as argument to all API
>> functions where it's required.
>>
>> I can do that, if there are no objections. What do you think?
>
> Sure. I have no objection.
Done in r1758183.
--
Ivan Zhakov
Received on 2016-08-29 10:06:52 CEST