user32/tests: Don't test function directly when reporting GetLastError()

André Hentschel nerv at dawncrow.de
Tue Jan 4 15:58:35 CST 2011


---
 dlls/user32/tests/clipboard.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/user32/tests/clipboard.c b/dlls/user32/tests/clipboard.c
index a0d0fe9..5b0f17e 100644
--- a/dlls/user32/tests/clipboard.c
+++ b/dlls/user32/tests/clipboard.c
@@ -68,8 +68,8 @@ static void test_ClipboardOwner(void)
     ok(OpenClipboard(hWnd1), "OpenClipboard failed\n");
 
     SetLastError(0xdeadbeef);
-    ok(!OpenClipboard(hWnd2) &&
-       (GetLastError() == 0xdeadbeef || GetLastError() == ERROR_ACCESS_DENIED),
+    ret = OpenClipboard(hWnd2);
+    ok(!ret && (GetLastError() == 0xdeadbeef || GetLastError() == ERROR_ACCESS_DENIED),
        "OpenClipboard should fail without setting last error value, or with ERROR_ACCESS_DENIED, got error %d\n", GetLastError());
 
     SetLastError(0xdeadbeef);
@@ -79,8 +79,8 @@ static void test_ClipboardOwner(void)
     ok(GetClipboardOwner() == hWnd1, "clipboard should be owned by %p, not by %p\n", hWnd1, GetClipboardOwner());
 
     SetLastError(0xdeadbeef);
-    ok(!OpenClipboard(hWnd2) &&
-       (GetLastError() == 0xdeadbeef || GetLastError() == ERROR_ACCESS_DENIED),
+    ret = OpenClipboard(hWnd2);
+    ok(!ret && (GetLastError() == 0xdeadbeef || GetLastError() == ERROR_ACCESS_DENIED),
        "OpenClipboard should fail without setting last error valuei, or with ERROR_ACCESS_DENIED, got error %d\n", GetLastError());
 
     ret = CloseClipboard();
-- 

Best Regards, André Hentschel



More information about the wine-patches mailing list