[PATCH] winspool: Use unsigned type to avoid a compiler warning

Detlef Riekenberg wine.dev at web.de
Sun Nov 4 10:57:54 CST 2007


---
 dlls/winspool.drv/info.c |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c
index e560f6a..cc4169b 100644
--- a/dlls/winspool.drv/info.c
+++ b/dlls/winspool.drv/info.c
@@ -145,7 +145,7 @@ static struct list monitor_handles = LIS
 static monitor_t * pm_localport;
 
 static opened_printer_t **printer_handles;
-static int nb_printer_handles;
+static UINT nb_printer_handles;
 static LONG next_job_id = 1;
 
 static DWORD (WINAPI *GDI_CallDeviceCapabilities16)( LPCSTR lpszDevice, LPCSTR lpszPort,
@@ -1563,11 +1563,9 @@ static opened_printer_t *get_opened_prin
 
     EnterCriticalSection(&printer_handles_cs);
 
-    if ((idx <= 0) || (idx > nb_printer_handles))
-        goto end;
-
-    ret = printer_handles[idx - 1];
-end:
+    if ((idx > 0) && (idx <= nb_printer_handles)) {
+        ret = printer_handles[idx - 1];
+    }
     LeaveCriticalSection(&printer_handles_cs);
     return ret;
 }
-- 
1.4.1


--=-jyGuRiYXimHND8p+orfO--




More information about the wine-patches mailing list