That is my reading as well - +1.

> On 30 Jun 2016, at 18:38, Jim Jagielski <j...@jagunet.com> wrote:
> 
> +1
> 
>> On Jun 30, 2016, at 11:38 AM, Stefan Eissing <stefan.eiss...@greenbytes.de> 
>> wrote:
>> 
>> We now set exactly the same callback right before in line 709. If we had 
>> more than one callback, we would not have to specify NULL, but restore any 
>> previous callback there was, right?
>> 
>> But that is all just theoretical, as You and Yann already stated.
>> 
>>> Am 30.06.2016 um 17:26 schrieb Yann Ylavic <ylavic....@gmail.com>:
>>> 
>>> On Thu, Jun 30, 2016 at 5:05 PM, Ruediger Pluem <rpl...@apache.org> wrote:
>>>> 
>>>> Is there a reson why we use ssl_callback_SSLVerify instead of NULL like we 
>>>> do in asimilar situation below?
>>>> IMHO we do not want to change the callback here to whatever it may set.
>>>> I agree that in practice there won't be any difference right now, since we 
>>>> only have one callback.
>>> 
>>> I agree that if/when we have multiple callback possibilities, we
>>> should set NULL here, but also above where we force the new mode.
>>> 
>>> Nothing to worry about for now, though.
>>> 
>>> Regards,
>>> Yann.
>> 
> 
> 

Reply via email to