Re: [PATCH] Resolve issue #4647 on trunk (v4)

2016-10-14 Thread Stefan
On 10/14/2016 4:05 PM, Stefan Sperling wrote: > On Fri, Oct 14, 2016 at 03:35:48PM +0200, Stefan wrote: >> Thanks for taking the time to review the patch, Ivan (and also stsp). >> After talking to stsp, here's a different approach to solve the issue >> without the need for adding a new parameter

Re: [PATCH] Resolve issue #4647 on trunk (v4)

2016-10-14 Thread Stefan Sperling
On Fri, Oct 14, 2016 at 03:35:48PM +0200, Stefan wrote: > Thanks for taking the time to review the patch, Ivan (and also stsp). > After talking to stsp, here's a different approach to solve the issue > without the need for adding a new parameter to the public API. > > (all tests pass on Windows

[PATCH] Resolve issue #4647 on trunk (v4)

2016-10-14 Thread Stefan
On 10/14/2016 12:19 PM, Ivan Zhakov wrote: > On 14 October 2016 at 00:29, Stefan wrote: >> On 10/13/2016 11:38 AM, Stefan wrote: >>> On 10/13/2016 11:08 AM, Stefan wrote: On 10/10/2016 11:39 PM, Stefan wrote: > On 10/10/2016 6:12 PM, Ivan Zhakov wrote: >> On 10