Re: [PATCH v2 0/2] MR1851: winhttp: Only read server reply in send_request() if the whole request is available.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/winhttp/request.c:
> {
>      BOOL async_mode = request->connect->hdr.flags & WINHTTP_FLAG_ASYNC;
>      DWORD ret, size, query, status;
> -    BOOL return_to_send;
>
> TRACE( "request state %d.\n", request->state );
>
> -    if (request->state >= REQUEST_RESPONSE_STATE_RESPONSE_RECEIVED)
> +    switch (request->state)
>      {
> -        ret = ERROR_WINHTTP_INCORRECT_HANDLE_STATE;
> -        goto done;
> -    }
> +        case REQUEST_RESPONSE_RECURSIVE_REQUEST:
> Could you remove one level of indentation in the switch?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1851#note_19911