Re: [PATCH v4 0/1] MR3518: wininet: unify timeout values closer to hInternet

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/wininet/internet.c:
> -
>    if (!lpwhh)
> -
>    {
> -
>        SetLastError(ERROR_INTERNET_INCORRECT_HANDLE_TYPE);
> -
>        return FALSE;
> -
>    }
> -
>    if (!lpBuffer || dwBufferLength != sizeof(ULONG))
> -
>    {
> -
>        SetLastError(ERROR_INVALID_PARAMETER);
> -
>        ret = FALSE;
> -
>    }
> -
>    else
> -
>    {
> -
>        lpwhh->connect_timeout = *(ULONG *)lpBuffer;
> -
>    }
> -
>    break;
> - }

Do we still need it? I think that with implementation in both `set_global_option` and `INET_SetOption`, we could simply remove those cases from `InternetSetOptionW` fallback.