Hans Leidekker : wininet: Get rid of the clear parameter to HTTP_GetResponseHeaders.
Alexandre Julliard
julliard at winehq.org
Thu Apr 25 14:25:18 CDT 2013
Module: wine
Branch: master
Commit: 2145b637fd4f374b0ecc23ef5ffe5abb0d558164
URL: http://source.winehq.org/git/wine.git/?a=commit;h=2145b637fd4f374b0ecc23ef5ffe5abb0d558164
Author: Hans Leidekker <hans at codeweavers.com>
Date: Thu Apr 25 10:14:21 2013 +0200
wininet: Get rid of the clear parameter to HTTP_GetResponseHeaders.
---
dlls/wininet/http.c | 19 ++++++++-----------
1 files changed, 8 insertions(+), 11 deletions(-)
diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c
index c910c59..5a58ac9 100644
--- a/dlls/wininet/http.c
+++ b/dlls/wininet/http.c
@@ -207,7 +207,7 @@ static CRITICAL_SECTION_DEBUG critsect_debug =
};
static CRITICAL_SECTION authcache_cs = { &critsect_debug, -1, 0, 0, 0, 0 };
-static BOOL HTTP_GetResponseHeaders(http_request_t *req, BOOL clear);
+static BOOL HTTP_GetResponseHeaders(http_request_t *req);
static DWORD HTTP_ProcessHeader(http_request_t *req, LPCWSTR field, LPCWSTR value, DWORD dwModifier);
static LPWSTR * HTTP_InterpretHttpHeader(LPCWSTR buffer);
static DWORD HTTP_InsertCustomHeader(http_request_t *req, LPHTTPHEADERW lpHdr);
@@ -4076,7 +4076,7 @@ static DWORD HTTP_SecureProxyConnect(http_request_t *request)
if (res != ERROR_SUCCESS)
return res;
- responseLen = HTTP_GetResponseHeaders( request, TRUE );
+ responseLen = HTTP_GetResponseHeaders( request );
if (!responseLen)
return ERROR_HTTP_INVALID_HEADER;
@@ -4873,7 +4873,7 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders,
INTERNET_SendCallback(&request->hdr, request->hdr.dwContext,
INTERNET_STATUS_RECEIVING_RESPONSE, NULL, 0);
- responseLen = HTTP_GetResponseHeaders(request, TRUE);
+ responseLen = HTTP_GetResponseHeaders(request);
/* FIXME: We should know that connection is closed before sending
* headers. Otherwise wrong callbacks are executed */
if(!responseLen && reusing_connection) {
@@ -5067,7 +5067,7 @@ static DWORD HTTP_HttpEndRequestW(http_request_t *request, DWORD dwFlags, DWORD_
INTERNET_SendCallback(&request->hdr, request->hdr.dwContext,
INTERNET_STATUS_RECEIVING_RESPONSE, NULL, 0);
- responseLen = HTTP_GetResponseHeaders(request, TRUE);
+ responseLen = HTTP_GetResponseHeaders(request);
if (!responseLen)
res = ERROR_HTTP_HEADER_NOT_FOUND;
@@ -5722,7 +5722,7 @@ static void HTTP_clear_response_headers( http_request_t *request )
* TRUE on success
* FALSE on error
*/
-static INT HTTP_GetResponseHeaders(http_request_t *request, BOOL clear)
+static INT HTTP_GetResponseHeaders(http_request_t *request)
{
INT cbreaks = 0;
WCHAR buffer[MAX_REPLY_LEN];
@@ -5742,6 +5742,9 @@ static INT HTTP_GetResponseHeaders(http_request_t *request, BOOL clear)
if(!request->netconn)
goto lend;
+ /* clear old response headers (eg. from a redirect response) */
+ HTTP_clear_response_headers( request );
+
NETCON_set_timeout( request->netconn, FALSE, request->receive_timeout );
do {
/*
@@ -5751,12 +5754,6 @@ static INT HTTP_GetResponseHeaders(http_request_t *request, BOOL clear)
if (!read_line(request, bufferA, &buflen))
goto lend;
- /* clear old response headers (eg. from a redirect response) */
- if (clear) {
- HTTP_clear_response_headers( request );
- clear = FALSE;
- }
-
rc += buflen;
MultiByteToWideChar( CP_ACP, 0, bufferA, buflen, buffer, MAX_REPLY_LEN );
/* check is this a status code line? */
More information about the wine-cvs
mailing list