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.
>>
>
>