Re: [PATCH 0/1] MR3583: wldap32: Map LDAP_OPT_AUTO_RECONNECT to LDAP_OPT_RESTART and add tests.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/wldap32/option.c:
>      return WLDAP32_LDAP_SUCCESS;
>  }
> - case WLDAP32_LDAP_OPT_AUTO_RECONNECT:
> - {
> -
>    BOOL *on = value;
> -
>    ret = ldap_get_option( CTX(ld), LDAP_OPT_RESTART, value );
> -
>    *on = !!*on;
> -
>    return ret;

We probably shouldn't touch 'value' on failure.