29 Nov
2022
29 Nov
'22
12:03 p.m.
On Mon Nov 28 12:28:46 2022 +0000, Jacek Caban wrote:
Implementing it like that means that we will send `INTERNET_STATUS_RECEIVING_RESPONSE` on each `InternetReadFile` call that reads from cache, is that intended? No, but `read_req_file` is placed here in case of asynchronous requests when contentPos is greater than current cache size.
And `read_req_file` is called first in `HTTPREQ_ReadFile` before sending `INTERNET_STATUS_RECEIVING_RESPONSE` and calling `HTTPREQ_Read`, so it is unlikely that we are receiving `INTERNET_STATUS_RECEIVING_RESPONSE` and getting data entirely from cache without reading HTTP stream if I understand it right. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1268#note_17843