Detlef Riekenberg : winspool: Use unsigned type to avoid a compiler warning .

Alexandre Julliard julliard at winehq.org
Wed Nov 7 06:46:01 CST 2007


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

Author: Detlef Riekenberg <wine.dev at web.de>
Date:   Sun Nov  4 17:57:54 2007 +0100

winspool: Use unsigned type to avoid a compiler warning.

---

 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 c4381d4..adb1b63 100644
--- a/dlls/winspool.drv/info.c
+++ b/dlls/winspool.drv/info.c
@@ -145,7 +145,7 @@ static struct list monitor_handles = LIST_INIT( monitor_handles );
 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,
@@ -1566,11 +1566,9 @@ static opened_printer_t *get_opened_printer(HANDLE hprn)
 
     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;
 }




More information about the wine-cvs mailing list