Paul Vriens : winspool/tests: Make default_printer a global variable.

Alexandre Julliard julliard at wine.codeweavers.com
Tue Dec 12 08:42:08 CST 2006


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

Author: Paul Vriens <paul.vriens.wine at gmail.com>
Date:   Tue Dec 12 10:22:32 2006 +0100

winspool/tests: Make default_printer a global variable.

---

 dlls/winspool.drv/tests/info.c |   15 +++------------
 1 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/dlls/winspool.drv/tests/info.c b/dlls/winspool.drv/tests/info.c
index 8f3bdd0..786ea3f 100644
--- a/dlls/winspool.drv/tests/info.c
+++ b/dlls/winspool.drv/tests/info.c
@@ -55,6 +55,8 @@ struct monitor_entry {
     CHAR  dllname[32];
 };
 
+static LPSTR   default_printer = NULL;
+
 /* report common behavior only once */
 static DWORD report_deactivated_spooler = 1;
 #define RETURN_ON_DEACTIVATED_SPOOLER(res) \
@@ -70,7 +72,6 @@ static DWORD report_deactivated_spooler
 
 static LPSTR find_default_printer(VOID)
 {
-    static  LPSTR   default_printer = NULL;
     static  char    buffer[DEFAULT_PRINTER_SIZE];
     DWORD   needed;
     DWORD   res;
@@ -1218,7 +1219,6 @@ static void test_OpenPrinter(void)
 {
     PRINTER_DEFAULTSA   defaults;
     HANDLE              hprinter;
-    LPSTR               default_printer;
     DWORD               res;
     DWORD               size;
     CHAR                buffer[DEFAULT_PRINTER_SIZE];
@@ -1311,7 +1311,7 @@ static void test_OpenPrinter(void)
 
 
     /* Get Handle for the default Printer */
-    if ((default_printer = find_default_printer()))
+    if (default_printer)
     {
         hprinter = (HANDLE) MAGIC_DEAD;
         SetLastError(MAGIC_DEAD);
@@ -1386,7 +1386,6 @@ static void test_OpenPrinter(void)
 static void test_SetDefaultPrinter(void)
 {
     DWORD   res;
-    LPSTR   default_printer;
     DWORD   size = DEFAULT_PRINTER_SIZE;
     CHAR    buffer[DEFAULT_PRINTER_SIZE];
     CHAR    org_value[DEFAULT_PRINTER_SIZE];
@@ -1395,8 +1394,6 @@ static void test_SetDefaultPrinter(void)
     if (!pSetDefaultPrinterA)  return;
 	/* only supported on win2k and above */
 
-    default_printer = find_default_printer();
-
     /* backup the original value */
     org_value[0] = '\0';
     SetLastError(MAGIC_DEAD);
@@ -1486,14 +1483,12 @@ static void test_SetDefaultPrinter(void)
 
 static void test_GetPrinterDriver(void)
 {
-    LPSTR default_printer;
     HANDLE hprn;
     BOOL ret;
     BYTE *buf;
     INT level;
     DWORD needed, filled;
 
-    default_printer = find_default_printer();
     if (!default_printer)
     {
         trace("There is no default printer installed, skiping the test\n");
@@ -1599,12 +1594,10 @@ static void test_DEVMODE(const DEVMODE *
 
 static void test_DocumentProperties(void)
 {
-    LPSTR default_printer;
     HANDLE hprn;
     LONG dm_size, ret;
     DEVMODE *dm;
 
-    default_printer = find_default_printer();
     if (!default_printer)
     {
         trace("There is no default printer installed, skiping the test\n");
@@ -1662,8 +1655,6 @@ static void test_EnumPrinters(void)
 
 START_TEST(info)
 {
-    LPSTR   default_printer;
-
     hwinspool = GetModuleHandleA("winspool.drv");
     pGetDefaultPrinterA = (void *) GetProcAddress(hwinspool, "GetDefaultPrinterA");
     pSetDefaultPrinterA = (void *) GetProcAddress(hwinspool, "SetDefaultPrinterA");




More information about the wine-cvs mailing list