Add a couple of missing bits to the SetWindowLong test

Dmitry Timoshkov dmitry at codeweavers.com
Tue Aug 1 09:05:42 CDT 2006


Hello,

Changelog:
    Add a couple of missing bits to the SetWindowLong test.

--- cvs/hq/wine/dlls/user/tests/win.c	Tue Aug 01 12:36:16 2006
+++ wine/dlls/user/tests/win.c	Tue Aug 01 13:48:11 2006
@@ -3831,6 +3831,7 @@ static int check_error(DWORD actual, DWO
 static void test_SetWindowLong(void)
 {
     LONG_PTR retval;
+    WNDPROC old_window_procW;
 
     SetLastError(0xdeadbeef);
     retval = SetWindowLongPtr(NULL, GWLP_WNDPROC, 0);
@@ -3854,11 +3855,15 @@ static void test_SetWindowLong(void)
         "SetWindowLongPtr on invalid window proc shouldn't have changed the value returned by GetWindowLongPtr, instead of changing it to 0x%x\n", retval);
     ok(!IsWindowUnicode(hwndMain), "hwndMain shouldn't be Unicode\n");
 
+    old_window_procW = (WNDPROC)GetWindowLongPtrW(hwndMain, GWLP_WNDPROC);
     SetLastError(0xdeadbeef);
     retval = SetWindowLongPtrW(hwndMain, GWLP_WNDPROC, 0);
     if (GetLastError() != ERROR_CALL_NOT_IMPLEMENTED)
     {
         ok(GetLastError() == 0xdeadbeef, "SetWindowLongPtr shouldn't have set the last error, instead of setting it to %ld\n", GetLastError());
+        ok(retval != 0, "SetWindowLongPtr error %ld\n", GetLastError());
+        ok((WNDPROC)retval == old_window_procW,
+            "SetWindowLongPtr on invalid window proc shouldn't have changed the value returned by GetWindowLongPtr, instead of changing it to 0x%x\n", retval);
         ok(IsWindowUnicode(hwndMain), "hwndMain should now be Unicode\n");
 
         /* set it back to ANSI */





More information about the wine-patches mailing list