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

[Alexandre Julliard]

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

Author: Detlef Riekenberg 
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;
 }