Re: [PATCH 0/2] MR4925: winhttp: Get output synchronously whenever possible in WinHttpQueryDataAvailable / WinHttpReadData.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/winhttp/request.c:
>  return count;
>
> }
>
> -static BOOL skip_async_queue( struct request *request )
> +static BOOL skip_async_queue( struct request *request, BOOL *data_available )
>  {
> - return request->hdr.recursion_count < 3 && (end_of_read_data( request ) || query_data_ready( request ));
> - *data_available = end_of_read_data( request ) || query_data_ready( request );

It's counterintuitive that *data_available is set to true when end_of_read_data() returns true. Perhaps we can find a better name, or pull that condition out of skip_async_queue()?