Re: svn commit: r1730617 - /subversion/trunk/subversion/libsvn_repos/log.c

2016-02-18 Thread Stefan Fuhrmann
On 17.02.2016 15:33, Ivan Zhakov wrote: On 16 February 2016 at 00:47, wrote: Author: stefan2 Date: Mon Feb 15 21:47:00 2016 New Revision: 1730617 URL: http://svn.apache.org/viewvc?rev=1730617&view=rev Log: Continue work on the svn_repos_get_logs4 to svn_repos_get_logs5 migration: Switch the l

Re: svn commit: r1730389 - /subversion/trunk/subversion/libsvn_repos/log.c

2016-02-18 Thread Stefan Fuhrmann
On 16.02.2016 11:30, Ivan Zhakov wrote: On 16 February 2016 at 08:47, Stefan Fuhrmann wrote: On 15.02.2016 09:39, Ivan Zhakov wrote: On 14 February 2016 at 22:25, wrote: Author: stefan2 Date: Sun Feb 14 19:25:12 2016 New Revision: 1730389 URL: http://svn.apache.org/viewvc?rev=1730389&vie

Re: svn commit: r1730617 -/subversion/trunk/subversion/libsvn_repos/log.c

2016-02-18 Thread Stefan Fuhrmann
On 15.02.2016 23:45, Bert Huijben wrote: Are you sure the pool arguments are in the right order here? The usual order is result_pool, scratch_pool while most of the calls here appear to use the opposite order. The pool usage is correct but somewhat confusing. r1731160 fixes that. -- Stefan^2.