Andrew Riedi : user32: Uncomment and explain a test in test_DestroyCursor() .

Alexandre Julliard julliard at winehq.org
Mon Dec 10 08:54:24 CST 2007


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

Author: Andrew Riedi <andrewriedi at gmail.com>
Date:   Sun Dec  9 23:05:29 2007 -0800

user32: Uncomment and explain a test in test_DestroyCursor().

---

 dlls/user32/tests/cursoricon.c |   11 +++++++++--
 1 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/dlls/user32/tests/cursoricon.c b/dlls/user32/tests/cursoricon.c
index 9858e32..6be44c9 100644
--- a/dlls/user32/tests/cursoricon.c
+++ b/dlls/user32/tests/cursoricon.c
@@ -411,9 +411,16 @@ static void test_DestroyCursor(void)
 
     SetCursor(NULL);
 
-    /* Trying to destroy the cursor properly fails now for some reason with ERROR_INVALID_CURSOR_HANDLE */
+    /* Trying to destroy the cursor properly fails now with
+     * ERROR_INVALID_CURSOR_HANDLE.  This happens because we called
+     * DestroyCursor() 2+ times after calling SetCursor().  The calls to
+     * GetCursor() and SetCursor(NULL) in between make no difference. */
     ret = DestroyCursor(cursor);
-    /* ok(ret, "DestroyCursor failed, GetLastError=%d\n", GetLastError()); */
+    todo_wine {
+        ok(!ret, "DestroyCursor succeeded.\n");
+        error = GetLastError();
+        ok(error == ERROR_INVALID_CURSOR_HANDLE, "Last error: 0x%08x\n", error);
+    }
 
     DeleteObject(cursorInfo.hbmMask);
     DeleteObject(cursorInfo.hbmColor);




More information about the wine-cvs mailing list