Andrew Talbot : wininet: Avoid signed-unsigned integer comparisons.

Alexandre Julliard julliard at winehq.org
Wed Mar 13 17:10:07 CDT 2013


Module: wine
Branch: master
Commit: ae77b3d368519b5518dbe485dfac2e8123b5a477
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ae77b3d368519b5518dbe485dfac2e8123b5a477

Author: Andrew Talbot <andrew.talbot at talbotville.com>
Date:   Tue Mar 12 22:28:49 2013 +0000

wininet: Avoid signed-unsigned integer comparisons.

---

 dlls/wininet/http.c          |    4 ++--
 dlls/wininet/internet.c      |    6 +++---
 dlls/wininet/netconnection.c |    4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c
index 498e848..a019095 100644
--- a/dlls/wininet/http.c
+++ b/dlls/wininet/http.c
@@ -4050,7 +4050,7 @@ static WORD HTTP_ParseWkday(LPCWSTR day)
                                      { 't','h','u',0 },
                                      { 'f','r','i',0 },
                                      { 's','a','t',0 }};
-    int i;
+    unsigned int i;
     for (i = 0; i < sizeof(days)/sizeof(*days); i++)
         if (!strcmpiW(day, days[i]))
             return i;
@@ -4313,7 +4313,7 @@ static WORD HTTP_ParseWeekday(LPCWSTR day)
                                      { 't','h','u','r','s','d','a','y',0 },
                                      { 'f','r','i','d','a','y',0 },
                                      { 's','a','t','u','r','d','a','y',0 }};
-    int i;
+    unsigned int i;
     for (i = 0; i < sizeof(days)/sizeof(*days); i++)
         if (!strcmpiW(day, days[i]))
             return i;
diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c
index 384516c..b6e6948 100644
--- a/dlls/wininet/internet.c
+++ b/dlls/wininet/internet.c
@@ -2836,7 +2836,7 @@ BOOL WINAPI InternetSetOptionW(HINTERNET hInternet, DWORD dwOption,
     case INTERNET_OPTION_PER_CONNECTION_OPTION: {
         INTERNET_PER_CONN_OPTION_LISTW *con = lpBuffer;
         LONG res;
-        int i;
+        unsigned int i;
         proxyinfo_t pi;
 
         INTERNET_LoadProxySettings(&pi);
@@ -2950,7 +2950,7 @@ BOOL WINAPI InternetSetOptionA(HINTERNET hInternet, DWORD dwOption,
                                    wbuffer, wlen );
         break;
     case INTERNET_OPTION_PER_CONNECTION_OPTION: {
-        int i;
+        unsigned int i;
         INTERNET_PER_CONN_OPTION_LISTW *listW;
         INTERNET_PER_CONN_OPTION_LISTA *listA = lpBuffer;
         wlen = sizeof(INTERNET_PER_CONN_OPTION_LISTW);
@@ -3019,7 +3019,7 @@ BOOL WINAPI InternetSetOptionA(HINTERNET hInternet, DWORD dwOption,
         if (dwOption == INTERNET_OPTION_PER_CONNECTION_OPTION)
         {
             INTERNET_PER_CONN_OPTION_LISTW *list = wbuffer;
-            int i;
+            unsigned int i;
             for (i = 0; i < list->dwOptionCount; ++i) {
                 INTERNET_PER_CONN_OPTIONW *opt = list->pOptions + i;
                 switch (opt->dwOption) {
diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c
index 3564282..c33f45a 100644
--- a/dlls/wininet/netconnection.c
+++ b/dlls/wininet/netconnection.c
@@ -477,7 +477,7 @@ static CRITICAL_SECTION init_ssl_cs = { &init_ssl_cs_debug, -1, 0, 0, 0, 0 };
 static DWORD init_openssl(void)
 {
 #ifdef SONAME_LIBCRYPTO
-    int i;
+    unsigned int i;
 
     if(OpenSSL_ssl_handle)
         return ERROR_SUCCESS;
@@ -730,7 +730,7 @@ void NETCON_unload(void)
     }
     if (ssl_locks)
     {
-        int i;
+        unsigned int i;
         for (i = 0; i < num_ssl_locks; i++)
         {
             ssl_locks[i].DebugInfo->Spare[0] = 0;




More information about the wine-cvs mailing list