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