ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

winspool.drv: Make sure to use return values (LLVM/Clang).

This commit is contained in:
Austin English 2011-02-04 23:08:04 -08:00 committed by Alexandre Julliard
parent 5da40467c8
commit 60c11fe037
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
dlls/winspool.drv/tests/info.c
View File

@ -372,6 +372,7 @@ static void test_AddMonitor(void)
mi2a.pEnvironment = entry->env; mi2a.pEnvironment = entry->env;
SetLastError(MAGIC_DEAD); SetLastError(MAGIC_DEAD);
res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a);
ok(res, "AddMonitor error %d\n", GetLastError());
/* NT: ERROR_INVALID_PARAMETER, 9x: ERROR_PRIVILEGE_NOT_HELD */ /* NT: ERROR_INVALID_PARAMETER, 9x: ERROR_PRIVILEGE_NOT_HELD */
} }
@ -1937,6 +1938,7 @@ static void test_SetDefaultPrinter(void)
org_value[0] = '\0'; org_value[0] = '\0';
SetLastError(MAGIC_DEAD); SetLastError(MAGIC_DEAD);
res = GetProfileStringA("windows", "device", NULL, org_value, size); res = GetProfileStringA("windows", "device", NULL, org_value, size);
ok(res, "GetProfileString error %d\n", GetLastError());
/* first part: with the default Printer */ /* first part: with the default Printer */
SetLastError(MAGIC_DEAD); SetLastError(MAGIC_DEAD);
@ -2011,11 +2013,13 @@ static void test_SetDefaultPrinter(void)
/* restore the original value */ /* restore the original value */
res = pSetDefaultPrinterA(default_printer); /* the nice way */ res = pSetDefaultPrinterA(default_printer); /* the nice way */
ok(res, "SetDefaultPrinter error %d\n", GetLastError());
WriteProfileStringA("windows", "device", org_value); /* the old way */ WriteProfileStringA("windows", "device", org_value); /* the old way */
buffer[0] = '\0'; buffer[0] = '\0';
SetLastError(MAGIC_DEAD); SetLastError(MAGIC_DEAD);
res = GetProfileStringA("windows", "device", NULL, buffer, size); res = GetProfileStringA("windows", "device", NULL, buffer, size);
ok(res, "GetProfileString error %d\n", GetLastError());
ok(!lstrcmpA(org_value, buffer), "'%s' (expected '%s')\n", buffer, org_value); ok(!lstrcmpA(org_value, buffer), "'%s' (expected '%s')\n", buffer, org_value);
} }
@ -2305,6 +2309,7 @@ static void test_GetPrinter(void)
{ {
DWORD double_needed; DWORD double_needed;
ret = pGetPrinterW(hprn, level, NULL, 0, &double_needed); ret = pGetPrinterW(hprn, level, NULL, 0, &double_needed);
ok(!ret, "level %d: GetPrinter error %d\n", level, GetLastError());
ok(double_needed == needed, "level %d: GetPrinterA returned different size %d than GetPrinterW (%d)\n", level, needed, double_needed); ok(double_needed == needed, "level %d: GetPrinterA returned different size %d than GetPrinterW (%d)\n", level, needed, double_needed);
} }
@ -2313,6 +2318,7 @@ static void test_GetPrinter(void)
SetLastError(0xdeadbeef); SetLastError(0xdeadbeef);
filled = -1; filled = -1;
ret = GetPrinter(hprn, level, buf, needed, &filled); ret = GetPrinter(hprn, level, buf, needed, &filled);
ok(ret, "level %d: GetPrinter error %d\n", level, GetLastError());
ok(needed == filled, "needed %d != filled %d\n", needed, filled); ok(needed == filled, "needed %d != filled %d\n", needed, filled);
if (level == 2) if (level == 2)
@ -2540,6 +2546,7 @@ static void test_GetPrinterDriver(void)
{ {
DWORD double_needed; DWORD double_needed;
ret = pGetPrinterDriverW(hprn, NULL, level, NULL, 0, &double_needed); ret = pGetPrinterDriverW(hprn, NULL, level, NULL, 0, &double_needed);
ok(!ret, "level %d: GetPrinterDriver error %d\n", level, GetLastError());
ok(double_needed == needed, "GetPrinterDriverA returned different size %d than GetPrinterDriverW (%d)\n", needed, double_needed); ok(double_needed == needed, "GetPrinterDriverA returned different size %d than GetPrinterDriverW (%d)\n", needed, double_needed);
} }