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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3518#note_72540